Bug 148422

Summary: Assertion failed: (rSData.empty() || rSData[0].m_bRedlineMoved || (rSData[0].m_nRedlineCount == rDoc.getIDocumentRedlineAccess().GetRedlineTable().size())), function SetSaveData, file undobj.cxx, line 1487.
Product: LibreOffice Reporter: Telesto <telesto>
Component: WriterAssignee: Not Assigned <libreoffice-bugs>
Status: NEW ---    
Severity: normal CC: michael.stahl
Priority: medium    
Version: 7.4.0.0 alpha0+   
Hardware: All   
OS: All   
See Also: https://bugs.documentfoundation.org/show_bug.cgi?id=133942
Whiteboard:
Crash report or crash signature: Regression By:
Bug Depends on:    
Bug Blocks: 105537    
Attachments: BT with symbols
bt with debug symbols

Description Telesto 2022-04-06 12:17:11 UTC 
Description:
Assertion failed: (rSData.empty() || rSData[0].m_bRedlineMoved || (rSData[0].m_nRedlineCount == rDoc.getIDocumentRedlineAccess().GetRedlineTable().size())), function SetSaveData, file undobj.cxx, line 1487.

Steps to Reproduce:
1. Open attachment 178254 [details]
2. Place cursor after 'grit'
3. Press backspace until grit is being deleted
4. Place cursor after ' put' 
5. press backspace until ' put' being deleted
6. press and hold CTRL+Z -> Assert

Actual Results:
Assert

Expected Results:
No assert


Reproducible: Always


User Profile Reset: No



Additional Info:
Version: 7.4.0.0.alpha0+ / LibreOffice Community
Build ID: fbf739198aa7f02975d531521c6525073783c7f1
CPU threads: 8; OS: Mac OS X 12.2.1; UI render: Skia/Metal; VCL: osx
Locale: nl-NL (nl_NL.UTF-8); UI: en-US
Calc: threaded
Comment 1 Telesto 2022-04-06 12:17:49 UTC 
Created attachment 179353 [details]
BT with symbols
Comment 2 Telesto 2022-04-06 12:20:09 UTC 
Simpeler way
1. Open the attached file
2. highlight 'grit' and 'put'
3. Press Undo
Comment 3 Julien Nabet 2022-04-06 18:15:06 UTC 
Created attachment 179359 [details]
bt with debug symbols

On pc Debian x86-64 with master sources updated today, I had an assertion applied first example but after some extra undoing.

About the second example, "Undo" button is disabled since LO considers there's nothing to undo. I suppose I missed something.
Comment 4 Telesto 2022-04-06 18:26:37 UTC 
(In reply to Julien Nabet from comment #3)
> Created attachment 179359 [details]
> bt with debug symbols
> 
> On pc Debian x86-64 with master sources updated today, I had an assertion
> applied first example but after some extra undoing.
> 
> About the second example, "Undo" button is disabled since LO considers
> there's nothing to undo. I suppose I missed something.

Apply Yellow Character Highlighting to 'grit' and 'put'. And next undo both changes..
Comment 5 Telesto 2022-04-08 20:34:08 UTC 
@Michael
There are still couple of rough edges (asserts) around redlining.. 
At this point I found three.. this one,  bug 148419 and bug 148461
Comment 6 Michael Stahl (allotropia) 2022-04-22 18:25:40 UTC 
this doesn't look like redlinehide-regression
Comment 7 Telesto 2022-05-20 18:43:29 UTC 
Another way
1. Open attachment 179679 [details]
2. Select "either contributed by volunteers, or funded"
3. Press backspace
4. Edit -> track changes -> Manage
5. Reject, reject, reject (so each entry one by one
6. Press and hold CTRL+Z
Comment 8 Telesto 2023-05-14 18:40:10 UTC 
Repro
Version: 7.6.0.0.alpha0+ (X86_64) / LibreOffice Community
Build ID: 066b23115c2a360507e306a88da572554daefab7
CPU threads: 8; OS: Mac OS X 12.6.3; UI render: Skia/Raster; VCL: osx
Locale: nl-NL (nl_NL.UTF-8); UI: en-US
Calc: threaded