Issue 58464

Summary: new chart: Import of charts from writer table needs to cope with wrong files written by OOo 2.0
Product: General Reporter: IngridvdM
Component: chartAssignee: michael.ruess
Status: CLOSED FIXED QA Contact: issues@graphics <issues>
Severity: Trivial    
Priority: P2 CC: issues
Version: 3.3.0 or older (OOo)Keywords: new_implementation
Target Milestone: ---   
Hardware: All   
OS: All   
Issue Type: TASK Latest Confirmation in: ---
Developer Difficulty: ---

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.