Bug 114169 - FILEOPEN XLSX Only the default, Box shape is interpreted in 3D column charts
Summary: FILEOPEN XLSX Only the default, Box shape is interpreted in 3D column charts
Status: RESOLVED FIXED
Alias: None
Product: LibreOffice
Classification: Unclassified
Component: Chart (show other bugs)
Version:
(earliest affected)
5.0 all versions
Hardware: All All
: medium normal
Assignee: Markus Mohrhard
URL:
Whiteboard: target:6.1.0 , target:6.0.0
Keywords: bibisected, bisected, regression
Depends on:
Blocks:
 
Reported: 2017-11-30 12:11 UTC by Gabor Kelemen (allotropia)
Modified: 2017-12-17 16:19 UTC (History)
3 users (show)

See Also:
Crash report or crash signature:

Attachments
Example file made with Excel 2013 (46.03 KB, application/vnd.openxmlformats-officedocument.spreadsheetml.sheet)
2017-11-30 12:11 UTC, Gabor Kelemen (allotropia) 		Details
The example file in Excel and Calc side by side (134.03 KB, image/png)
2017-11-30 12:14 UTC, Gabor Kelemen (allotropia) 		Details
The example file saved by LO6 alpha (33.71 KB, application/vnd.openxmlformats-officedocument.spreadsheetml.sheet)
2017-11-30 12:15 UTC, Gabor Kelemen (allotropia) 		Details
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Gabor Kelemen (allotropia) 2017-11-30 12:11:36 UTC 
Created attachment 138118 [details]
Example file made with Excel 2013

Microsoft Excel offers the following Column shapes for 3D Column Charts: Box, Full Pyramid, Partial Pyramid, Cylinder, Full Cone and Partial Cone. LibreOffice Calc displays all of these as box shaped columns.

Steps to reproduce:
1. Create a new spreadsheet in LibreOffice Excel 2013.
2. Create 3D column chart based on any values.
3. Change the shape of the 3D column chart from the default Box to Full Pyramid, Partial Pyramid, Cylinder, Full Cone or Partial Cone.
4. Save the file as XLSX.
5. Open the file in LibreOffice Calc.

Actual results:
The 3D column chart will be displayed with box shaped columns.

Expected results:
The shape of the columns should be preserved.
Comment 1 Gabor Kelemen (allotropia) 2017-11-30 12:14:36 UTC 
Created attachment 138119 [details]
The example file in Excel and Calc side by side
Comment 2 Gabor Kelemen (allotropia) 2017-11-30 12:15:30 UTC 
Created attachment 138120 [details]
The example file saved by LO6 alpha
Comment 3 raal 2017-12-02 11:17:32 UTC 
I can confirm with Version: 6.1.0.0.alpha0+
Build ID: aa28b54ab568fb7160095bb4a2fec338b1398b43
CPU threads: 4; OS: Linux 4.4; UI render: default; VCL: gtk3; 

regression - works in 4.1.
Comment 4 raal 2017-12-02 11:49:40 UTC 
This seems to have begun at the below commit.
Adding Cc: to Markus Mohrhard ; Could you possibly take a look at this one?
Thanks

There are only 'skip'ped commits left to test.
The first bad commit could be any of: 976c469b22d9c0cd4c82eb6b70806013bdac3c79 84f786cde77ac35cf564f36e687cf2d04fb6a953
We cannot bisect more!

# only skipped commits left to test
# possible first bad commit: [84f786cde77ac35cf564f36e687cf2d04fb6a953] source-hash-4c3cfe60ac05d5404e3f2a92fff7af2d2f38628a
# possible first bad commit: [976c469b22d9c0cd4c82eb6b70806013bdac3c79] source-hash-5fa0adf25cfcb71129e05e7218cde46874306c15


author	Markus Mohrhard <markus.mohrhard@googlemail.com>	2015-04-20 06:21:17 +0200
committer	Markus Mohrhard <markus.mohrhard@googlemail.com>	2015-04-21 06:45:26 +0200
commit 4c3cfe60ac05d5404e3f2a92fff7af2d2f38628a (patch)
tree 47d5141591f3c93c138d5face9968c287f41ad16
parent 5fa0adf25cfcb71129e05e7218cde46874306c15 (diff)
MSO 2007 vs OOXML default values
includes:

fix data label delete element handling
Comment 5 Markus Mohrhard 2017-12-16 19:14:40 UTC 
No idea what i thought when I wrote that code but it is obviously wrong.
Comment 6 Gabor Kelemen (allotropia) 2017-12-17 16:19:04 UTC 
Thanks!

Patch: https://gerrit.libreoffice.org/#/c/46614/