anne/lyx_mirror
1
0
Fork 0
mirror of https://git.lyx.org/repos/lyx.git synced 2024-09-19 14:30:32 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

Fix data loss in user guide conversion to 2.0.x

Browse Source 
When doing the lyx2lyx round trip of the 2.0.8 user guide from format
413 -> 474 -> 413 you do not get a zero diff. The most important problem is
caused by the conversion of the argument insets to the old syntax: This
conversion adds an additional empty line (harmless), and it destroys the
document structure if the first inset in e.g. a subsection is not an argument
inset, but e.g. an index or label inset.
The fix is quite easy: Ensure that the paragraph begin is set to the first
argument inset.
(cherry picked from commit 28bde98d76)

Conflicts:
	status.21x
This commit is contained in:
Georg Baum 2014-04-27 18:06:34 +02:00 committed by Richard Heck
parent a87c56b5ad
commit 602a37af48
1 changed files with 12 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

15
lib/lyx2lyx/lyx_2_1.py
View File

@ -1552,12 +1552,18 @@ def revert_latexargs(document):
continue
parbeg = parent[1]
parend = parent[2]
realparbeg = parent[3]
# Collect all arguments in this paragraph
# Do not set realparbeg to parent[3], since this does not work if we
# have another inset (e.g. label or index) before the first argument
# inset (this is the case in the user guide of LyX 2.0.8)
realparbeg = -1
# Collect all arguments in this paragraph
realparend = parend
for p in range(parbeg, parend):
m = rx.match(document.body[p])
if m:
if realparbeg < 0:
# This is the first argument inset
realparbeg = p
val = int(m.group(1))
j = find_end_of_inset(document.body, p)
# Revert to old syntax
@ -1573,8 +1579,11 @@ def revert_latexargs(document):
del document.body[p : j + 1]
if p >= realparend:
break
if realparbeg < 0:
# No argument inset found
realparbeg = parent[3]
# Now sort the arg insets
subst = [""]
subst = []
for f in sorted(args):
subst += args[f]
del args[f]