Issue 58464 - new chart: Import of charts from writer table needs to cope with wrong files written by OOo 2.0
Summary: new chart: Import of charts from writer table needs to cope with wrong files ...
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: michael.ruess
QA Contact: issues@graphics
URL:
Keywords: new_implementation
Depends on:
Blocks:
 
Reported: 2005-11-25 21:26 UTC by IngridvdM
Modified: 2013-02-24 21:19 UTC (History)
1 user (show)

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

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this issue.
Description IngridvdM 2005-11-25 21:26:59 UTC 
This Issue is cause by Issue 58458. The previous Office version export a corrupt
data range, which leads to empty files when loaded whith the new chart.
Comment 1 IngridvdM 2005-11-25 21:28:02 UTC 
accepted
Comment 2 IngridvdM 2005-11-29 18:39:36 UTC 
->TL: I already fixed the detection of the table name. But the cellrange still
has to be shifted  one row up. Please look at the new version of unochart.cxx (v
1.1.2.22) in line 565. The cellrange given in aRangeRepresentation needs to be
shifted. To get my fixes you need an uptodate xmloff and chart2 module. There is
a new build on the way.

Furthermore the invalid cell range seems to lead to memorycorruption in writer
(try to load a second document with some charts for example->crash). This should
be fixed also. If the writer is not able to cope with invalid ranges it is
necessary to clip a given range to a valid range in createDataSource I think.
Comment 3 thomas.lange 2005-12-06 09:31:21 UTC 
.
Comment 4 thomas.lange 2005-12-06 12:52:45 UTC 
Fixed in CWS chart2mst3.

Files changed:
sw:
- unochart.cxx
Comment 5 IngridvdM 2006-01-18 15:01:36 UTC 
Ready for verification in CWS chart2mst3 Milestone4.

re-open issue and reassign to kla@openoffice.org
Comment 6 IngridvdM 2006-01-18 15:01:42 UTC 
reassign to kla@openoffice.org
Comment 7 IngridvdM 2006-01-18 15:01:47 UTC 
reset resolution to FIXED
Comment 8 IngridvdM 2006-03-22 07:55:59 UTC 
changed summary to start with 'new chart: ' to avoid confusion with regular
chart bugs
Comment 9 IngridvdM 2006-03-22 11:45:47 UTC 
changed issue type from defect to task to better reflect that this issue is only
a working issue for the chart reimplementation
Comment 10 bjoern.milcke 2006-06-23 10:02:35 UTC 
Changed target to 2.x
Comment 11 IngridvdM 2007-01-18 11:47:23 UTC 
changed target to 2.3
Comment 12 thomas.lange 2007-02-20 09:49:08 UTC 
TL->MRU: You need to ask IHA how to verify this one.
Comment 13 michael.ruess 2007-03-28 13:40:32 UTC 
Verified fix in CWS chart2mst3.
Comment 14 michael.ruess 2007-06-21 15:43:07 UTC 
Checked in 680m214.