Issue 58458 - Fileformat violation when saving Charts with data from Writer tables
Summary: Fileformat violation when saving Charts with data from Writer tables
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: jogi
QA Contact: issues@graphics
URL:
Keywords: oooqa
Depends on:
Blocks:
 
Reported: 2005-11-25 20:34 UTC by IngridvdM
Modified: 2013-02-24 21:22 UTC (History)
2 users (show)

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


Attachments
Example file with chart from writer table (11.32 KB, application/vnd.oasis.opendocument.text)
2005-11-25 20:36 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:34:22 UTC
Create a chart from a writer table.
Save the document and look at the content.xml of the chart subobject.
1) The data range given in 'table:cell-range-address' is lacking the name of the
table.
2) The range has a wrong offset and thus looses the first row.

In the example file the range is ".$A$2:.$C$7" but it should be "Table1.$A$1:.$C$6".
Comment 1 IngridvdM 2005-11-25 20:36:46 UTC
Created attachment 31814 [details]
Example file with chart from writer table
Comment 2 bjoern.milcke 2005-11-28 08:58:43 UTC
accepted
Comment 3 bjoern.milcke 2005-12-09 10:35:30 UTC
There is no easy fix for the table name. We could transport it somehow over the
old SomeData interface, however I don't know how to adapt it when the table name
changes. Apart the left over SomeData strings (3 and 4) are used for evil things
by writer and may thus overwrite the table name.

We have a workaround: The chart can retrieve the HierarchicalName and can pass
it to Writer. When Writer is asked to create a data source based on a malformed
string (no table-name) it can use the this name to retrieve the table name by
internal means.

I fixed the second problem with the row-indexes.
Comment 4 bjoern.milcke 2005-12-09 17:38:21 UTC
please verify in cws sch13

re-open issue and reassign to kla@openoffice.org
Comment 5 bjoern.milcke 2005-12-09 17:38:27 UTC
reassign to kla@openoffice.org
Comment 6 bjoern.milcke 2005-12-09 17:38:32 UTC
reset resolution to FIXED
Comment 7 bjoern.milcke 2005-12-09 17:48:13 UTC
BM->TK: Please verify. If you have problems verifying this (there is just a
change in the content.xml file), ask IHA to verify this bug or ask me for
explanations.

re-open issue and reassign to kla@openoffice.org
Comment 8 bjoern.milcke 2005-12-09 17:48:23 UTC
reassign to kla@openoffice.org
Comment 9 bjoern.milcke 2005-12-09 17:48:28 UTC
reset resolution to FIXED
Comment 10 bjoern.milcke 2005-12-20 12:36:35 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.


re-open issue and reassign to bm
Comment 11 bjoern.milcke 2005-12-20 12:36:47 UTC
reassign to bm
Comment 12 bjoern.milcke 2005-12-20 12:36:51 UTC
reset resolution to FIXED
Comment 13 bjoern.milcke 2005-12-20 12:38:01 UTC
reopened
Comment 14 bjoern.milcke 2005-12-20 12:38:39 UTC
accepted
Comment 15 bjoern.milcke 2006-05-19 17:07:09 UTC
Depends on Issue 60323, because we need a reliable version info in OLE objects.
Comment 16 bjoern.milcke 2006-06-23 14:04:50 UTC
Changed Target to 2.x
Comment 17 bjoern.milcke 2007-03-01 16:41:19 UTC
The chart OLE objects new export meta information. The issue will be fixed with
integration of the new chart.
Comment 18 IngridvdM 2007-03-01 21:06:55 UTC
changed target
Comment 19 bjoern.milcke 2007-03-02 12:03:28 UTC
Fixed in chart2mst3 Milestone 11_1.

->JSI: To test, save a Writer file containing a chart (create a new one, or make
sure the chart is really written by activating it once) with the new chart and
check that in the OLE object's content.xml (e.g. Object 1/content.xml) the
table:cell-range-address, and all chart:*-range-address contents contain the
table name, i.e. sth. like "Table1.$A1:.$A7" instead of just ".$A1:.$A7".
Comment 20 michael.ruess 2007-03-28 13:34:20 UTC
Verified fix in CWS chart2mst3.
Comment 21 michael.ruess 2007-03-28 13:37:30 UTC
uäh... put have edited wrong issue. Reopened.
Comment 22 michael.ruess 2007-03-28 13:38:10 UTC
Setting back to "fixed".
Comment 23 jogi 2007-05-08 07:36:35 UTC
ok in chart2mst3 #12_4 build (based on SRCm210)
The document is valid.
Comment 24 ace_dent 2008-05-16 03:28:47 UTC
This Issue is 'Verified' and not updated in 1yr+, so Closing.
A Closed Issue is a Happy Issue (TM).

Regards,
Andrew
 
Cleaning-up and Closing old Issues as part of:
~ The Grand Bug Squash, pre v3 ~