Bug 108022 - FILESAVE DOCX: One column chart is exported instead of two
Summary: FILESAVE DOCX: One column chart is exported instead of two
Status: VERIFIED FIXED
Alias: None
Product: LibreOffice
Classification: Unclassified
Component: Chart (show other bugs)
Version:
(earliest affected)
4.4 all versions
Hardware: All All
: high major
Assignee: Markus Mohrhard
URL:
Whiteboard: target:6.3.0 target:6.2.0.1 target:6.1.4
Keywords: dataLoss, filter:docx
Depends on:
Blocks: OOXML-Chart
  Show dependency treegraph
 
Reported: 2017-05-23 10:02 UTC by Gabor Kelemen (allotropia)
Modified: 2018-11-21 00:38 UTC (History)
2 users (show)

See Also:
Crash report or crash signature:


Attachments
Docx version of attachment 133470 (14.94 KB, application/vnd.openxmlformats-officedocument.wordprocessingml.document)
2017-05-23 10:02 UTC, Gabor Kelemen (allotropia)
Details
Screenshot of the original document in LO 5.3 and the docx in Word 2010 side by side (322.68 KB, image/png)
2017-05-23 10:04 UTC, Gabor Kelemen (allotropia)
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Gabor Kelemen (allotropia) 2017-05-23 10:02:31 UTC
Created attachment 133471 [details]
Docx version of attachment 133470 [details]

When saving attachment 133470 [details] as docx from LO 5.3 I get a docx file with one chart instead of two. 
The first chart completely disappears from the export.
Comment 1 Gabor Kelemen (allotropia) 2017-05-23 10:04:35 UTC
Created attachment 133472 [details]
Screenshot of the original document in LO 5.3 and the docx in Word 2010 side by side
Comment 2 Xisco Faulí 2017-05-23 10:26:34 UTC
The correct file is: attachment 133469 [details]

Confirmed in

- Version: 5.4.0.0.alpha1+
Build ID: 74d2e606fd3605fe0a585f596eaa215ae4e20d18
CPU Threads: 4; OS Version: Linux 4.8; UI Render: default; VCL: gtk3; 
Locale: en-US (ca_ES.UTF-8); Calc: group

- Version: 5.2.0.0.alpha1+
Build ID: 5b168b3fa568e48e795234dc5fa454bf24c9805e
CPU Threads: 4; OS Version: Linux 4.8; UI Render: default; 
Locale: ca-ES (ca_ES.UTF-8)

- Version: 5.0.0.0.alpha1+
Build ID: 0db96caf0fcce09b87621c11b584a6d81cc7df86
Locale: ca-ES (ca_ES.UTF-8)

- Version: 4.3.0.0.alpha1+
Build ID: c15927f20d4727c3b8de68497b6949e72f9e6e9e
Comment 3 QA Administrators 2018-05-26 02:42:08 UTC Comment hidden (obsolete)
Comment 4 Markus Mohrhard 2018-11-18 22:23:59 UTC
Based on a quick git blame while fixing the bug this might actually be a regression.
Comment 5 Commit Notification 2018-11-18 23:08:38 UTC
Markus Mohrhard committed a patch related to this issue.
It has been pushed to "master":

https://git.libreoffice.org/core/+/9ccdf4d8abf6ec6612c88ab65be25eb6be89b455%5E%21

tdf#108022, don't overwrite chart export information

It will be available in 6.3.0.

The patch should be included in the daily builds available at
https://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More
information about daily builds can be found at:
https://wiki.documentfoundation.org/Testing_Daily_Builds

Affected users are encouraged to test the fix and report feedback.
Comment 6 Commit Notification 2018-11-19 01:00:23 UTC
Markus Mohrhard committed a patch related to this issue.
It has been pushed to "libreoffice-6-2":

https://git.libreoffice.org/core/+/6436c8fd5f0da3de6d8af639432c9b4924cb417d%5E%21

tdf#108022, don't overwrite chart export information

It will be available in 6.2.0.1.

The patch should be included in the daily builds available at
https://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More
information about daily builds can be found at:
https://wiki.documentfoundation.org/Testing_Daily_Builds

Affected users are encouraged to test the fix and report feedback.
Comment 7 Xisco Faulí 2018-11-19 09:16:27 UTC
Verified in

Version: 6.2.0.0.beta1+
Build ID: 268364e35100b559f42d8c02b930c5cca1c84be7
CPU threads: 4; OS: Linux 4.15; UI render: default; VCL: gtk3; 
Locale: ca-ES (ca_ES.UTF-8); UI-Language: en-US
Calc: threaded

@Markus Mohrhard, thanks for fixing this!!
Comment 8 Commit Notification 2018-11-19 15:02:35 UTC
Markus Mohrhard committed a patch related to this issue.
It has been pushed to "libreoffice-6-1":

https://git.libreoffice.org/core/+/21467c1c0fea676a33c7b7e75648947efea18fcb%5E%21

tdf#108022, don't overwrite chart export information

It will be available in 6.1.4.

The patch should be included in the daily builds available at
https://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More
information about daily builds can be found at:
https://wiki.documentfoundation.org/Testing_Daily_Builds

Affected users are encouraged to test the fix and report feedback.
Comment 9 Commit Notification 2018-11-21 00:38:25 UTC
Markus Mohrhard committed a patch related to this issue.
It has been pushed to "master":

https://git.libreoffice.org/core/+/0abce68927f45673b73d8f806e108a51a36fb543%5E%21

add test for tdf108022

It will be available in 6.3.0.

The patch should be included in the daily builds available at
https://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More
information about daily builds can be found at:
https://wiki.documentfoundation.org/Testing_Daily_Builds

Affected users are encouraged to test the fix and report feedback.