Bug 38745

Summary:	redlining / undo document breakage / corruption
Product:	LibreOffice	Reporter:	Michael Meeks <michael.meeks>
Component:	Writer	Assignee:	Michael Stahl <mst.fdo>
Status:	RESOLVED FIXED	QA Contact:
Severity:	critical
Priority:	medium	CC:	cno, courrier.oou.fr.mjk, detective.conan.1412, jeffdchang, karlsen-masur, LibreOffice, luis.antolin.cano, sbergman, yfjiang
Version:	3.4.1 RC2	Keywords:	regression
Hardware:	Other
OS:	All
Whiteboard:	target:3.4.6 target:3.5.1 target:3.6.0beta0
i915 platform:		i915 features:
Bug Depends on:
Bug Blocks:	39721, 35673
Attachments:	test file

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 karlsen-masur 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 karlsen-masur 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 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:

http://cgit.freedesktop.org/libreoffice/core/commit/?id=13424b43c25389e303774c3fb2f2beb3e20ceae5

Comment 12 Stephan Bergmann 2012-02-07 02:57:40 UTC

cherry-picked to -3-4 (637d80a4aebf2b65a855283a68e6dd72e9d6de30) and -3-5 (de998662158e3654ac14469989d27ce963cce935)

Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.