Bug 38745 - redlining / undo document breakage / corruption
Summary: redlining / undo document breakage / corruption
Alias: None
Product: LibreOffice
Classification: Unclassified
Component: Writer (show other bugs)
(earliest affected)
3.4.1 RC2
Hardware: Other All
: medium critical
Assignee: Michael Stahl (allotropia)
Whiteboard: target:3.4.6 target:3.5.1 target:3.6....
Keywords: regression
Depends on:
Blocks: mab3.4 39721
  Show dependency treegraph
Reported: 2011-06-28 06:42 UTC by Michael Meeks
Modified: 2012-02-07 04:05 UTC (History)
9 users (show)

See Also:
Crash report or crash signature:

test file (11.24 KB, application/vnd.oasis.opendocument.text)
2011-06-28 06:43 UTC, Michael Meeks

Note You need to log in before you can comment on or make changes to this bug.
Description Michael Meeks 2011-06-28 06:42:24 UTC
+ Load the attached document
+ double click on the second heading line 'Frobnication of the ...'
+ hit 'delete'
+ hit ctrl-z to undo

notice the lack of undoing ;-) and the cursor jumping to the top of the document.

I guess it is some poisonous interaction with not showing red-lining. If I Edit->Changes->Show the redlining, then it behaves as I would expect.

I got some nasty crash working on red-lining this document that (I imagine) is related to some funky underlying document tree corruption around this sort of thing.
Comment 1 Michael Meeks 2011-06-28 06:43:57 UTC
Created attachment 48516 [details]
test file
Comment 2 Jeffrey 2011-06-28 07:47:31 UTC
Reproduced on LibreOffice 3.4  340m1(Build:12) on KDE OpenSuse (Linux). Agree with Michael. Seems like once the red highlighting appears, it erases the history.
Comment 3 Cor Nouws 2011-06-29 02:04:28 UTC
relation to Bug 38329 - Undo broken for several actions ?
Comment 4 Korrawit Pruegsanusak 2011-06-29 07:37:19 UTC
related to bug 37584 - track changes: deleted text lost ?
Comment 5 K.M. 2011-07-06 02:08:36 UTC
I can confirm this for

LibreOffice 3.4.1, OOO340m1 (Build:103)

using Windowmaker on openSuSE 11.4 (Linux - 64 bit)

having "Edit->Changes->Record" checked/enabled, but "Edit->Changes->Show" disabled/unchecked.

I even got the crash after some more undoing, redoing actions.

If other combinations are used for those track changes options, it works fine.

I suggest adding this to "Bug 38993 - [TASK]: Bugfixes required for enterprise deployment LibO 3.4.2" since it is an often used functionality in enterprise use and it cause data losses because of the immanent crash.
Comment 6 Rainer Bielefeld Retired 2011-07-16 01:46:51 UTC
[Reproducible] with "LibreOffice 3.4.1  - WIN7  Home Premium (64bit) German UI [OOO340m1 (Build:103)]", but a quick test to create an own sample document failed.
Still a problem in Master.

I did not check whether DUP
Comment 7 Rainer Bielefeld Retired 2011-07-23 07:50:28 UTC
Worked for me with 3.3.3 Portable, so REGRESSION
Comment 8 K.M. 2011-11-02 06:43:48 UTC
Seems that LO 3.4.4rc2 (LibreOffice 3.4.4, OOO340m1 (Build:402)) got rid of the crash when messing with the redo/undo actions. Rest of the described bug behavior is still happening with this version of LO.
Comment 9 lantolin 2011-11-15 07:34:06 UTC
Reproduced at LO 3.4.4 final at Ubuntu 11.10 by following the steps indicated and the test2.odt.
Comment 10 Björn Michaelsen 2011-12-23 13:22:41 UTC
Since all new unconfirmed bugs start in state UNCONFIRMED now and old unconfirmed bugs were moved to NEEDINFO with a explanatory comment, all bugs promoted above those bug states to NEW and later are automatically confirmed making the CONFIRMED whiteboard status redundant. Thus it will be removed.
Comment 11 Michael Stahl (allotropia) 2012-02-07 02:14:12 UTC
ouch, that must be the stupidest regression i've ever put into OOo:
when the UndoImpl method of SwRedlineUndoDelete is called,
the deleted text is not in the body but in the hidden redline
section of the nodes array, because:
the stack guard is immediately destroyed, before calling the
UndoImpl method it was supposed to cover!

regression from CWS undoapi

fixed on master:

Comment 12 Stephan Bergmann 2012-02-07 02:57:40 UTC
cherry-picked to -3-4 (637d80a4aebf2b65a855283a68e6dd72e9d6de30) and -3-5 (de998662158e3654ac14469989d27ce963cce935)