From: Richard Heck <rgheck@comcast.net>
Date: Tue, 2 Nov 2010 15:24:49 +0000 (+0000)
Subject: Fix lyx2lyx problem related to bug #1881.
X-Git-Tag: 2.0.0~2114
X-Git-Url: https://git.lyx.org/gitweb/?a=commitdiff_plain;h=580a984e2ca469f8c83c461f4b8e743b18ce9f47;p=features.git

Fix lyx2lyx problem related to bug #1881.

It is dangerous (i.e., foolish) to rely upon assumptions about exactly
how many lines there will be between one structure and another structure
in a LyX file. No such things should be hardcoded into lyx2lyx. You must
ALWAYS search for what you want, e.g., the beginning of the layout in
which some particular inset is contained.



git-svn-id: svn://svn.lyx.org/lyx/lyx-devel/trunk@35998 a592a061-630c-0410-9148-cb99ea01b6c8
---

diff --git a/lib/lyx2lyx/lyx_2_0.py b/lib/lyx2lyx/lyx_2_0.py
index 779a0c5f07..bf891087a1 100644
--- a/lib/lyx2lyx/lyx_2_0.py
+++ b/lib/lyx2lyx/lyx_2_0.py
@@ -2353,32 +2353,69 @@ def revert_diagram(document):
     return
 
 
-def convert_bibtexClearpage(document):
+def convert_bibtex_clearpage(document):
   " insert a clear(double)page bibliographystyle if bibtotoc option is used "
+
+  i = find_token(document.header, '\\papersides', 0)
+  if i == -1:
+    document.warning("Malformed LyX document: Can't find papersides definition.")
+    return
+  sides = int(document.header[i][12])
+
+  j = 0
   while True:
-    i = find_token(document.header, '\\papersides', 0)
-    if i == -1:
-      document.warning("Malformed LyX document: Can't find papersides definition.")
-      return
-    else:
-      sides = int(document.header[i][12])
-    # only act of there is the option "bibtotoc"
-    j = find_token(document.body, 'options "bibtotoc', 0)
+    j = find_token(document.body, "\\begin_inset CommandInset bibtex", j)
     if j == -1:
       return
+
+    k = find_end_of_inset(document.body, j)
+    if k == -1:
+      document.warning("Can't find end of Bibliography inset at line " + str(j))
+      j += 1
+      continue
+
+    # only act if there is the option "bibtotoc"
+    m = find_token(document.body, 'options', j)
+    if m == -1 or m > k:
+      document.warning("Can't find options for bibliography inset at line " + str(j))
+      j = k
+      continue
+    
+    optline = document.body[m]
+    idx = optline.find("bibtotoc")
+    if idx == -1:
+      j = k
+      continue
+    
+    # so we want to insert a new page right before the paragraph that
+    # this bibliography thing is in. we'll look for it backwards.
+    lay = j - 1
+    while lay >= 0:
+      if document.body[lay].startswith("\\begin_layout"):
+        break
+      lay -= 1
+
+    if lay < 0:
+      document.warning("Can't find layout containing bibliography inset at line " + str(j))
+      j = k
+      continue
+
     subst1 = '\\begin_layout Standard\n' \
       + '\\begin_inset Newpage clearpage\n' \
-      + '\end_inset\n\n\n' \
-      + '\end_layout\n'
+      + '\\end_inset\n\n\n' \
+      + '\\end_layout\n'
     subst2 = '\\begin_layout Standard\n' \
       + '\\begin_inset Newpage cleardoublepage\n' \
-      + '\end_inset\n\n\n' \
-      + '\end_layout\n'
+      + '\\end_inset\n\n\n' \
+      + '\\end_layout\n'
     if sides == 1:
-      document.body.insert(j -5, subst1)
+      document.body.insert(lay, subst1)
+      document.warning(subst1)
     else:
-      document.body.insert(j -5, subst2)
-    return
+      document.body.insert(lay, subst2)
+      document.warning(subst2)
+
+    j = k
 
 
 ##
@@ -2442,7 +2479,7 @@ convert = [[346, []],
            [399, [convert_mathdots]],
            [400, [convert_rule]],
            [401, []],
-           [402, [convert_bibtexClearpage]],
+           [402, [convert_bibtex_clearpage]],
            [403, [convert_flexnames]],
            [404, [convert_prettyref]]
 ]