Bug 168737 - FILEOPEN DOCX Overlapping redlines: multiple Accept All / Reject All needed
Summary: FILEOPEN DOCX Overlapping redlines: multiple Accept All / Reject All needed
Status: ASSIGNED
Alias: None
Product: LibreOffice
Classification: Unclassified
Component: Writer (show other bugs)
Version:
(earliest affected)
26.2.0.0 alpha0+ master
Hardware: All All
: medium normal
Assignee: Miklos Vajna
URL:
Whiteboard:
Keywords: filter:docx
Depends on:
Blocks: DOCX-Track-Changes
  Show dependency treegraph
 
Reported: 2025-10-08 07:40 UTC by Gabor Kelemen (Collabora)
Modified: 2025-12-10 09:16 UTC (History)
2 users (show)

See Also:
Crash report or crash signature:


Attachments
Example file from Word 2016 (14.20 KB, application/vnd.openxmlformats-officedocument.wordprocessingml.document)
2025-10-08 07:40 UTC, Gabor Kelemen (Collabora)
Details
The example file after pressing Accept All (69.52 KB, image/png)
2025-10-08 07:41 UTC, Gabor Kelemen (Collabora)
Details
The example file after pressing Reject All (71.21 KB, image/png)
2025-10-08 07:42 UTC, Gabor Kelemen (Collabora)
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Gabor Kelemen (Collabora) 2025-10-08 07:40:49 UTC
Created attachment 203195 [details]
Example file from Word 2016

Attached example file was made in Word to imitate a change tracked editing scenario: inserting text, formatting it, then deleting part of it as another user.

This file opens fine in Writer, but to accept or reject all changes, the user needs to press the Accept All or Reject All buttons several times, which is contrary to their supposed function.

1. Open attached file
2. In the Writer sidebar go to the Manage Changes section
3/a. Press Accept All
-> a deletion is still there 
3/b. Press Reject All
-> Formatting, insertion and deletion are still there. You need to press 4 times the Reject all to really reject all tracked changes in this file.

Version: 26.2.0.0.alpha0+ (X86_64) / LibreOffice Community
Build ID: 708abf24664a9a1aaeac452bcc8436026030e74e
CPU threads: 14; OS: Windows 10 X86_64 (build 19045); UI render: Skia/Raster; VCL: win
Locale: de-DE (hu_HU); UI: en-US
Calc: threaded

This was differently bad in 25.8:
Accept All did not leave changes behind, but it gave incorrect results
Reject All left similar amount of changes behind, but crashed on the second click.
Comment 1 Gabor Kelemen (Collabora) 2025-10-08 07:41:26 UTC
Created attachment 203196 [details]
The example file after pressing Accept All
Comment 2 Gabor Kelemen (Collabora) 2025-10-08 07:42:07 UTC
Created attachment 203197 [details]
The example file after pressing Reject All
Comment 3 Sarah Lim 2025-11-03 09:35:16 UTC
Confirmed that the bug is present in master.

Version: 26.2.0.0.alpha0+ (X86_64) / LibreOffice Community
Build ID: 620(Build:0)
CPU threads: 12; OS: Windows 11 X86_64 (build 26100); UI render: Skia/Vulkan; VCL: win
Locale: en-US (en_FI); UI: en-GB
Calc: threaded
Comment 4 Buovjaga 2025-11-08 17:41:33 UTC
(In reply to Gabor Kelemen (Collabora) from comment #0)
> This was differently bad in 25.8:
> Accept All did not leave changes behind, but it gave incorrect results
> Reject All left similar amount of changes behind, but crashed on the second
> click.

The behaviour change was commit 55640f9f0a4741f8e4b5b98096af822cee71da2c
Comment 5 Miklos Vajna 2025-12-10 09:16:29 UTC
(In reply to Buovjaga from comment #4)
> The behaviour change was commit 55640f9f0a4741f8e4b5b98096af822cee71da2c

Thanks for the bisect, let me see.