Bug 136426 - Filesave DOC: Missing last chart in MSO (seen in LO)
Summary: Filesave DOC: Missing last chart in MSO (seen in LO)
Status: NEW
Alias: None
Product: LibreOffice
Classification: Unclassified
Component: Writer (show other bugs)
Version:
(earliest affected)
4.2.6.3 release
Hardware: All All
: medium normal
Assignee: Not Assigned
URL:
Whiteboard:
Keywords: bibisected, bisected, dataLoss, filter:doc
Depends on:
Blocks: DOC
  Show dependency treegraph
 
Reported: 2020-09-03 09:15 UTC by Timur
Modified: 2020-10-27 10:37 UTC (History)
3 users (show)

See Also:
Crash report or crash signature:


Attachments
Sample source DOCX (68.74 KB, application/vnd.openxmlformats-officedocument.wordprocessingml.document)
2020-09-03 09:15 UTC, Timur
Details
Sample saved with LO 420 beta (50.00 KB, application/msword)
2020-09-03 10:15 UTC, Timur
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Timur 2020-09-03 09:15:42 UTC
Created attachment 165075 [details]
Sample source DOCX

Attached sample. I'm not sure if this is a valid DOCX, it was first created in LO and then resaved in MSO, it opens fine both in MSO and LO, so I guess it is. 

If opened in LO and saved as DOC, opens fine in LO but 4th chart is missing in MSO. Seen in screenshot attachment 165072 [details].
Repro with LO 7.1+ and previously LO 5.0.6. No repro LO 5.0 beta. Regression. 
Save as DOCX is OK.
Comment 1 Timur 2020-09-03 09:42:21 UTC Comment hidden (me-too)
Comment 2 Timur 2020-09-03 10:04:31 UTC
Correction: repro LO 5.0 beta and 4.3, no repro 4.2.0 beta.
I tried to bibisect in Linux 42max but failed, oldest saves no charts, latest saved 3 charts. I assumed there should be 4 charts somewhere in between, but I didn't see it.
Comment 3 Timur 2020-09-03 10:15:31 UTC
Created attachment 165079 [details]
Sample saved with LO 420 beta
Comment 4 Buovjaga 2020-09-04 13:25:03 UTC
(In reply to Timur from comment #2)
> Correction: repro LO 5.0 beta and 4.3, no repro 4.2.0 beta.
> I tried to bibisect in Linux 42max but failed, oldest saves no charts,
> latest saved 3 charts. I assumed there should be 4 charts somewhere in
> between, but I didn't see it.

You could try checking out latest and then git checkout HEAD~1000 or similar to go backwards.
Comment 5 Timur 2020-09-14 10:17:49 UTC
I did try, in vain, couldn't find the working LO in Linux. 
Regardless of bibisect, please confirm the bug.
Comment 6 Buovjaga 2020-09-23 13:55:18 UTC
Confirmed with MSO 2013

Version: 7.1.0.0.alpha0+ (x86)
Build ID: abcc4eb907661e07ad850ccce7eb06f129da4286
CPU threads: 4; OS: Windows 6.3 Build 9600; UI render: Skia/Raster; VCL: win
Locale: fi-FI (fi_FI); UI: en-US
Calc: threaded
Comment 7 Timur 2020-10-19 12:22:25 UTC Comment hidden (obsolete)
Comment 8 Aron Budea 2020-10-27 07:14:12 UTC
I bibisected the change in Windows to the following commit.
Probably not a regression in the traditional sense, but might be useful information nevertheless.

https://cgit.freedesktop.org/libreoffice/core/commit/?id=a56bfe734a7bed40a175e0eeb7e86e7006a83f83
author		Miklos Vajna <vmiklos@collabora.co.uk>	2013-11-22 16:34:18 +0100
committer	Miklos Vajna <vmiklos@collabora.co.uk>	2013-11-22 16:59:56 +0100

cp#1000016 DOCX/RTF import: fix RemoveLastParagraph() on Windows
Comment 9 Timur 2020-10-27 10:37:11 UTC
That's why I couldn't find working commit in Linux. 
This is commit d077117c6c82a4118fa6df15ffee3148db43bf74 (tag: source-hash-a56bfe734a7bed40a175e0eeb7e86e7006a83f83) from 4.3, but commit before also doesn't work. So this is Windows-only change which is not a regression. Thanks.