Issue 118954 - Chart data will lost after copy to different file
Summary: Chart data will lost after copy to different file
Status: CLOSED FIXED
Alias: None
Product: General
Classification: Code
Component: chart (show other issues)
Version: 3.3.0 or older (OOo)
Hardware: PC Windows XP
: P3 Critical (vote)
Target Milestone: ---
Assignee: Wang Lei
QA Contact:
URL:
Keywords: regression
Depends on:
Blocks: 118840
  Show dependency tree
 
Reported: 2012-02-22 07:17 UTC by Li Feng Wang
Modified: 2013-02-24 21:19 UTC (History)
8 users (show)

See Also:
Issue Type: DEFECT
Latest Confirmation in: ---
Developer Difficulty: ---
jsc: 3.4_release_blocker+

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 Li Feng Wang 2012-02-22 07:17:46 UTC 
reproduce steps:
1)new a chart in sheet1(spreadsheet)
2)Right Click sheet1 tab, choose "Move or copy" option in context menu.
3)Check "Copy" checkbox, and copy this sheet to another file
4)Check new pasted sheet, chart lost

Defect:
chart lost in step4

notes:
Chart will not lost when pasted to the same file.
Comment 1 zhao xia 2012-02-27 08:58:22 UTC 
Retested against AOO-dev build, Rev 1240836, couldn't reproduce.
Comment 2 Li Feng Wang 2012-02-27 09:07:11 UTC 
reproduce in AOO-dev build 1243392

in pasted sheet, chart data lost
Comment 3 zhao xia 2012-02-27 09:12:38 UTC 
Confirmed the detail steps, this problem can reproduce when copy and pasted to one existing file, if select "new document" in Copy/Move dialog, this problem wouldn't occurs.

Also, modify the problem to "Chat data" lost not chat lost.
Comment 4 zhao xia 2012-03-12 09:32:09 UTC 
Propose as release blocker
Comment 5 Wang Lei 2012-03-12 10:02:03 UTC 
This issue can not be reproduced in OOo3.3.0(OOO330m17 build:9551). It is a regression issue.
Comment 6 hanya 2012-03-13 05:39:10 UTC 
issue 118840 is duplicate of this
Comment 7 jolatt 2012-03-13 10:01:19 UTC 
The data aren't lost. They are only not visible. 
Double click on the chart, then menu-format-chart-type. Select the same or a new type and hit o.k. 

AOO 340m1(Build:9588) - Rev. 1299571
BS: Xubuntu 11.10
Comment 8 hanya 2012-03-13 16:55:48 UTC 
(In reply to comment #6)
> issue 118840 is duplicate of this
Sorry, this issue is not the same with issue 118840
issue 118840 is caused by copying chart from Calc to other type of document.
Comment 9 Wang Lei 2012-03-15 08:54:11 UTC 
I think I found the reason.

The reason is before adjusting source range of chart, it update the chart view which has wrong source range.

In OOo3.3, it does not need to update chart view when call ScDocument::TransferDrawPage(). There is some enhancement for drawlayer layer which make Chart view to update.

The solution is force chart view to be updated after adjusting source range of chart.

I will deliver the fix soon
Comment 10 Wang Lei 2012-03-16 06:01:49 UTC 
Fix this issue with revision 1301345
Comment 11 Oliver-Rainer Wittmann 2012-03-16 08:39:25 UTC 
marking issue as fixed.
Comment 12 Li Feng Wang 2012-03-27 02:50:11 UTC 
It's Ok on build r1303653 (developer snapshot)