Issue 58457 - Fileformat violation when saving Donut Charts
Summary: Fileformat violation when saving Donut Charts
Status: CLOSED FIXED
Alias: None
Product: General
Classification: Code
Component: chart (show other issues)
Version: 3.3.0 or older (OOo)
Hardware: All All
: P2 Trivial (vote)
Target Milestone: ---
Assignee: kla
QA Contact: issues@graphics
URL:
Keywords: ms_interoperability
: 58461 (view as issue list)
Depends on:
Blocks:
 
Reported: 2005-11-25 20:16 UTC by IngridvdM
Modified: 2013-02-24 21:20 UTC (History)
3 users (show)

See Also:
Issue Type: DEFECT
Latest Confirmation in: ---
Developer Difficulty: ---


Attachments
The example Donut created from columns (11.57 KB, application/vnd.oasis.opendocument.spreadsheet)
2005-11-25 20:20 UTC, IngridvdM
no flags Details
The example Bar Chart created from columns (10.34 KB, application/vnd.oasis.opendocument.spreadsheet)
2005-11-25 20:21 UTC, IngridvdM
no flags Details

Note You need to log in before you can comment on or make changes to this issue.
Description IngridvdM 2005-11-25 20:16:24 UTC
Open a calc document and type some numbers in 2 columns and 4 rows for example.
Create a donut chart from that data in columns (default).
Save the document and look at the content.xml file of the chart subobject.

For comparison create a bar chart from the same data with same options.

The property 'chart:series-source' is set to 'columns' in both files as expected.
The data writen to <table:table> subelement is almost identical. But the donut
chart pretends to have for columns
<table:table-column table:number-columns-repeated="4"/>
which is not true (instead a number of 2 is fitting to the written table).
Furthermore the number of series and Points is not fitting to the given
datatable seperated in columns as indicated in chart:series-source.

The settings for donut chart for rows or columns are mixed up in the gui which
leads to the wrong settings in the file. Please export the correct setting
'chart:series-source="rows"' for donut charts if the GUI setting wrongly states
'from columns' and vice versa.
Furthermore correct the wrong count for table columns repeat.

The import than has to be adapted also to cope with both version.
Comment 1 IngridvdM 2005-11-25 20:20:55 UTC
Created attachment 31812 [details]
The example Donut created from columns
Comment 2 IngridvdM 2005-11-25 20:21:43 UTC
Created attachment 31813 [details]
The example Bar Chart created from columns
Comment 3 bjoern.milcke 2005-11-28 08:58:21 UTC
accepted
Comment 4 bjoern.milcke 2005-12-15 16:06:31 UTC
Fixed.

ToDo: Adapt Build Id in SchXMLBuildInfo.hxx
Comment 5 bjoern.milcke 2005-12-20 12:37:41 UTC
Files created with OOo 2.0 or OOo 2.0.1 and saved with OOo 2.0.2 will get
corrupted due to a missing working fileformat versioning.  When a file is
created with OOo 2.0 and loaded with a version containing these fixes and saved
again, the files will get corrupted, because the OLE objects are just copied as
they are (saved with OOo 2.0), but the parent document is re-saved with the new
version.  As OLE objects have no own meta information, they inherit it from
their parent which in this case (the default) is wrong.
Comment 6 bjoern.milcke 2005-12-20 12:38:21 UTC
accepted
Comment 7 bjoern.milcke 2006-05-19 17:04:48 UTC
Depends on Issue 60323, because we need a reliable version info in OLE objects.
Comment 8 bjoern.milcke 2006-06-23 14:04:47 UTC
Changed Target to 2.x
Comment 9 IngridvdM 2006-08-25 19:25:52 UTC
Björn, I will take over.
Comment 10 IngridvdM 2006-08-25 19:39:57 UTC
Fixed in CWS chart2mst3.
Donuts are now saved according to the ODF standard 1.1. One donut series is
represented as one ring.
Wrong files from older versions are converted correctly.
Older versions of OpenOffice will not read the correct files correctly - so if
you need to work with donut charts, please update to the corrected version.

Todo: Update the build ID BUILDID_CHART2INTEGRATION in
xmloff/source/chart/SchXMLErrorBuildIds.hxx to the Build ID of integration of
the CWS chart2mst3.
Comment 11 IngridvdM 2006-08-25 19:44:55 UTC
removed dependency to separat import issue as import and export was handled here
together
Comment 12 IngridvdM 2006-08-25 19:48:12 UTC
*** Issue 58461 has been marked as a duplicate of this issue. ***
Comment 13 IngridvdM 2006-12-14 14:03:11 UTC
@Thomas: Fixed in CWS chart2mst3 milestone10, please verify.
Comment 14 IngridvdM 2007-01-18 11:55:46 UTC
changed target to 2.3
Comment 15 kla 2007-05-07 09:28:30 UTC
@Jsi: pls verify this issue
Comment 16 frank 2007-06-18 12:02:20 UTC
found integrated on master src680 m215