Apache OpenOffice (AOO) Bugzilla – Full Text Issue Listing |
Summary: | DTD: Unexpected Choice chart:bar for attribute chart:class | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | General | Reporter: | frank | ||||||
Component: | chart | Assignee: | michael.brauer | ||||||
Status: | CLOSED WONT_FIX | QA Contact: | issues@graphics <issues> | ||||||
Severity: | Trivial | ||||||||
Priority: | P2 | CC: | issues, michael.brauer, tony.galmiche.ooo | ||||||
Version: | 3.3.0 or older (OOo) | ||||||||
Target Milestone: | --- | ||||||||
Hardware: | All | ||||||||
OS: | All | ||||||||
Issue Type: | DEFECT | Latest Confirmation in: | --- | ||||||
Developer Difficulty: | --- | ||||||||
Issue Depends on: | |||||||||
Issue Blocks: | 75528, 75529 | ||||||||
Attachments: |
|
Description
frank
2007-03-29 14:29:47 UTC
targeted Created attachment 44020 [details]
bugdoc
. If we extend binfilter to understand the class attribute at data-series (to fix Issue 75528 and Issue 75529), we need to fix this. The transformation between OOo and ODF adds/removes a namespace "chart:" at the chart:class attribute of the chart:chart element. This transformation is missing for the chart:series element. When this issue is fixed, files in OOo (e.g. ".sxc") format created with StarOffice 7.0 (OOo 1.1) or StarOffice 8 (OOo 2) containing stock charts of types 3 or 4 (with volume) will not import correctly. And the other way round, that is files created with the fixed version will load incorretly in the aforementioned versions. Stock charts of type 1 or 2, and combi-charts with bars and lines will import correctly between the versions. ->MIB: What do you think about the impact this has? If we don't fix this issue, we have to add a namespace in binfilter to conform to the "non-conforming" OOo format that is currently implemented. Created attachment 44166 [details]
Fix for this bug in xmloff
(Note: The reason why the combi chart still works but the stock chart doesn't is, that the combi-chart prefers the "lines-used" property, but the stock chart import prefers the chart:class="chart:bar" (with the wrong namespace) over the property "stock-with-volume") This issue seems to risky to me to fix. It breaks old files and leads to new files that conform the DTD but will be read incorrectly by old versions. I don't really see the benefit. If this issue will be fixed, we will need an additional fix in binfilter in binfilter/bf_xmloff/source/chart/xmloff_SchXMLExport.cxx In the beginning of the file there is a commented-out macro defined ISSUE_75865_IS_FIXED. This macro has to be enabled to apply a different behaviour that conforms to the fix made in xmloff (see patch). (Addition to previous note: the macro will be available after integration of CWS OOF680/oasisrng02) ->FST: See my comments before for the reasoning to fix this. A fix exists (see attachment), but it breaks some OOo files (with stock-charts of type 3 and 4). Hi Michael, what's your point of view here ? If you vote for closing, please do so. Frank I also do not see any benefit in resolving that. |