From: Uwe Stöhr <uwestoehr@lyx.org>
Date: Sat, 26 May 2012 17:00:03 +0000 (+0200)
Subject: lyx2lyx/lyx_2_1.py: fix #8172
X-Git-Tag: 2.1.0beta1~1833
X-Git-Url: https://git.lyx.org/gitweb/?a=commitdiff_plain;h=7b680058756b4d80e741faa4bdafe894bc06360d;p=features.git

lyx2lyx/lyx_2_1.py: fix #8172
---

diff --git a/lib/lyx2lyx/lyx_2_1.py b/lib/lyx2lyx/lyx_2_1.py
index bbd9b3cd9f..84bf2bc507 100644
--- a/lib/lyx2lyx/lyx_2_1.py
+++ b/lib/lyx2lyx/lyx_2_1.py
@@ -347,27 +347,28 @@ def convert_use_packages(document):
     "use_xxx yyy => use_package xxx yyy"
     packages = ["amsmath", "esint", "mathdots", "mhchem", "undertilde"]
     for p in packages:
-        i = find_token(document.header, "\\use_%s" % p , 0)
+        i = find_token(document.header, "\\use_%s" % p, 0)
         if i != -1:
-            value = get_value(document.header, "\\use_%s" % p , i)
+            value = get_value(document.header, "\\use_%s" % p, i)
             document.header[i] = "\\use_package %s %s" % (p, value)
 
 
 def revert_use_packages(document):
     "use_package xxx yyy => use_xxx yyy"
-    packages = {"amsmath":"1", "esint":"1", "mathdots":"1", "mhchem":"1", "undertilde":"1"}
+    packages = ["amsmath", "esint", "mathdots", "mhchem", "undertilde"]
     # the order is arbitrary for the use_package version, and not all packages need to be given.
     # Ensure a complete list and correct order (important for older LyX versions and especially lyx2lyx)
-    j = -1
-    for p in packages.keys():
+    j = 0
+    for p in packages:
         regexp = re.compile(r'(\\use_package\s+%s)' % p)
-        i = find_re(document.header, regexp, 0)
+        i = find_re(document.header, regexp, j)
         if i != -1:
-            value = get_value(document.header, "\\use_package" , i).split()[1]
+            value = get_value(document.header, "\\use_package %s" % p, i).split()[1]
+            document.warning(str(value))
             del document.header[i]
             j = i
-    for (p, v) in packages.items():
-        document.header.insert(j, "\\use_%s %s"  % (p, value))
+            document.header.insert(j, "\\use_%s %s"  % (p, value))
+            document.warning(str(value))
         j = j + 1