Bug 89323 - undo option after entering values in table is not working in Libreoffice Impress
Summary: undo option after entering values in table is not working in Libreoffice Impress
Status: NEW
Alias: None
Product: LibreOffice
Classification: Unclassified
Component: Impress (show other bugs)
Version:
(earliest affected)
4.4.0.3 release
Hardware: x86 (IA32) All
: medium normal
Assignee: Not Assigned
URL:
Whiteboard:
Keywords: bibisected, bisected, regression
Depends on:
Blocks: Undo-Redo
  Show dependency treegraph
 
Reported: 2015-02-12 04:21 UTC by Sheeba Medoline
Modified: 2023-12-11 01:33 UTC (History)
4 users (show)

See Also:
Crash report or crash signature:


Attachments
Screenshot of Impress which displays the struck page after did undo option for many times (210.27 KB, image/jpeg)
2015-02-12 04:21 UTC, Sheeba Medoline
Details
steps to reproduce (71.05 KB, application/vnd.oasis.opendocument.text)
2015-02-21 09:07 UTC, raal
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Sheeba Medoline 2015-02-12 04:21:05 UTC
Created attachment 113328 [details]
Screenshot of Impress which displays the struck page after did undo option for many times

Actual Result:
After creating a table, undo option for table is not working randomly in Libreoffice Impress when doing the undo work repeatedly.

Expected Result:

After creating a table, undo option for table is not working randomly.
Even doing undo option repeatedly, it should undo the previous work.

Step 1: open Libre office Impress
Step 2: create new slides.
Step 3: create table
Step 4: click undo
Step 5: It is working for first few times
Step 6: When we repeat the same work, it is not working
Step 7: Page get struck and Libre Office get crashed but it can be recovered
Comment 1 Sheeba Medoline 2015-02-12 04:29:47 UTC
Actual Result:
After creating a table, undo option for table is not working randomly in Libreoffice Impress when doing the undo work repeatedly.

Expected Result:

After creating a table, undo option for table is not working randomly.
Even doing undo option repeatedly, it should undo the previous work.

Step 1: open Libre office Impress
Step 2: create new slides.
Step 3: create table
Step 4: enter values in cells
Step 5: click undo
Step 6: It is working for first few time(completely undo the entire table)
Step 7: When we repeat the same work , it is not working(undo few last entered values)
Step 8: Page get struck and Some times, Libre Office get crashed but it can be recovered
Comment 2 raal 2015-02-12 19:42:38 UTC
I can confirm with Version: 4.4.0.3
ID build: de093506bcdc5fafd9023ee680b8c60e3e0645d7
I can confirm with Version: 4.5.0.0.alpha0+
Build ID: 62969accf9c01b71b738424d4d643db8bfaed182
TinderBox: Linux-rpm_deb-x86_64@46-TDF, Branch:master, Time: 2015-02-08_23:22:32

to reproduce insert table into table. 
No crash or freeze reproduced. Cannot reproduce with LibreOffice 3.5.0 
Build ID: d6cde02 -> regression
Comment 3 Matthew Francis 2015-02-21 03:05:46 UTC
@raal: Could you clarify exactly what you reproduced and how?
I can't reproduce a crash on Linux, but I can reproduce the following issue:

1. Create a new presentation
2. Type a few characters of text into the slide body
3. Insert a table
4. Type a few characters into a table cell

Result:
- Actions are present in the undo stack for the body text insertion (2) / table text insertion (4), but not for the table insertion itself (3)


In the 43all bibisect tree, before

# possible first bad commit: [a0f20bc04a32a7791ba765d2de2f44f1b74033d1] source-hash-1de66ba440855050a794b3b2a8647c1b02c210b8
# possible first bad commit: [6848bca2a871fce1f3f28a8471eab4e5888ac9c9] source-hash-b45876bf0f2eeafba0a4f9f8f30cd4279eb2aa3e

the table insertion gets an undo stack entry, but it is wrongly ordered (before the slide body text insertion rather than after). The original issue of the wrongly ordered undo stack insertion is inherited from OOo (exists in OOo 3.3.0)


On the assumption that this is all the same issue, tagging this as bibisected for now - the above is at least a worsening of the original bug
Comment 4 Matthew Francis 2015-02-21 07:23:41 UTC
The point in which the table insert stops getting an undo stack entry at all in the sequence of comment 3 is the below commit

Adding Cc: to caolanm@redhat.com; could you possibly take a look at this? Thanks


commit 12a4200e8ff7f045efcc7e9d15a24b15b248c437
Author:     Armin Le Grand <alg@apache.org>
AuthorDate: Thu Aug 9 08:42:27 2012 +0000
Commit:     Caolán McNamara <caolanm@redhat.com>
CommitDate: Tue May 28 15:55:23 2013 +0100

    Related: #i120498# Enhanced Undo/Redo and user experience...
    
    when editing texts in graphic objects and/or tables
    
    (cherry picked from commit a096725779b210c7a2706e72596fd7c80e049fdd)
Comment 5 raal 2015-02-21 09:07:41 UTC
Created attachment 113572 [details]
steps to reproduce
Comment 6 Robinson Tryon (qubit) 2015-12-13 11:12:13 UTC Comment hidden (obsolete)
Comment 7 Miklos Vajna 2016-05-19 13:39:11 UTC
Adding Cc: to Armin.Le.Grand@me.com as well.
Comment 8 Xisco Faulí 2016-09-26 15:03:16 UTC
Adding Cc: to Armin Le Grand
Comment 9 QA Administrators 2018-04-22 02:32:03 UTC Comment hidden (obsolete)
Comment 10 QA Administrators 2020-04-22 03:40:57 UTC Comment hidden (obsolete)
Comment 11 Armin Le Grand 2022-01-04 14:41:39 UTC
This commit was developed for another code base, and not merged by me. For complex changes like this, side-effects are to be expected; sadly I dont't have the cycles to deal with all the fallout. Un-Ccing myself for the while.
Comment 12 Tex2002ans 2023-12-11 01:33:10 UTC
I CAN'T reproduce in:

Version: 7.6.4.1 (X86_64) / LibreOffice Community
Build ID: e19e193f88cd6c0525a17fb7a176ed8e6a3e2aa1
CPU threads: 8; OS: Windows 10.0 Build 19045; UI render: Skia/Raster; VCL: win
Locale: en-US (en_US); UI: en-US
Calc: CL threaded

- - -

I tried all sorts of insert table + type + undo steps:

- Comment #0, Comment #1, Comment #3, even the steps in Comment #5's document

and LO Impress undid everything perfectly fine.

(The undo table bug may have already been fixed in these past 7 years.)