Issue 75258 - New Chart: Renaming a sheet corrupts charts using data on it
Summary: New Chart: Renaming a sheet corrupts charts using data on it
Status: CLOSED FIXED
Alias: None
Product: General
Classification: Code
Component: chart (show other issues)
Version: 3.3.0 or older (OOo)
Hardware: All All
: P3 Trivial (vote)
Target Milestone: ---
Assignee: kla
QA Contact: issues@graphics
URL:
Keywords: new_implementation
Depends on:
Blocks:
 
Reported: 2007-03-09 15:58 UTC by bjoern.milcke
Modified: 2013-02-24 21:21 UTC (History)
2 users (show)

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

Attachments
Document with two charts that show the corruption on sheet rename (17.07 KB, application/vnd.oasis.opendocument.spreadsheet)
2007-03-09 16:01 UTC, bjoern.milcke 		no flags Details
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 bjoern.milcke 2007-03-09 15:58:50 UTC 
1. Load the attached document
2a. Rename the first sheet
=> The chart data is set incorrectly.
(It looks like the data would be set as rectangular region with wrong arguments).

2b. activate both charts once, so that they are loaded
3b. Rename the first sheet
=> The chart data is set differently but also incorrectly.

2c. Press F9
3c. Rename the first sheet
=> The charts are OK.

->NN: I am not sure why there are these differences. I think when F9 is pressed,
the charts have living models and the model objects are attached as change
listener and therefore get the new table name in the used ranges.
In the other cases the charts are loaded, but obviously the data with renamed
sheet is set like in the old chart as rectangular region
(XDataReceiver::setArguments). This, of course, does not always work, and also
might destroy formatting.
Comment 1 bjoern.milcke 2007-03-09 15:59:48 UTC 
Set keyword, CC kla.
Comment 2 bjoern.milcke 2007-03-09 16:01:19 UTC 
Created attachment 43622 [details]
Document with two charts that show the corruption on sheet rename
Comment 3 bjoern.milcke 2007-03-09 16:05:02 UTC 
Sorry, forgot a little detail: appears on CWS chart2mst3.
Comment 4 niklas.nebel 2007-03-19 16:06:10 UTC 
Fixed on CWS chart2mst3.
Comment 5 niklas.nebel 2007-03-20 10:48:16 UTC 
Available in CWS build, reassigning to QA for verification.
Comment 6 kla 2007-03-26 11:40:39 UTC 
verified
Comment 7 p9w.vu.31122010 2007-06-14 14:16:24 UTC 
verified in 680 m215, closing