Bug 111358 - EDITING: No visible chart when pasting from Calc into Impress/Writer (because of missing data)
Summary: EDITING: No visible chart when pasting from Calc into Impress/Writer (because...
Status: RESOLVED FIXED
Alias: None
Product: LibreOffice
Classification: Unclassified
Component: Impress (show other bugs)
Version:
(earliest affected)
6.0.0.0.alpha0+
Hardware: All All
: medium normal
Assignee: Not Assigned
URL:
Whiteboard:
Keywords: bibisected, bisected, regression
Depends on:
Blocks:
 
Reported: 2017-08-04 18:02 UTC by Telesto
Modified: 2017-08-07 07:56 UTC (History)
2 users (show)

See Also:
Crash report or crash signature:

Attachments
Example file (14.04 KB, application/vnd.oasis.opendocument.spreadsheet)
2017-08-04 18:03 UTC, Telesto 		Details
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Telesto 2017-08-04 18:02:41 UTC 
Description:
No visible chart when pasting from Calc into Impress/Writer; The data in column E isn't embedded into the object

Steps to Reproduce:
1. Open the attached file
2. Copy the chart
3. Open a new Writer/Impress document
4. Paste

Actual Results:  
The object is pasted, with legend but without a pie chart. The data in column E isn't embedded into the object

Expected Results:
Pie chart should be visible


Reproducible: Always

User Profile Reset: No

Additional Info:
Found in
Version: 6.0.0.0.alpha0+
Build ID: 46b9d35ceaba80ce73fb0b4b5a87dbdf0a674628
CPU threads: 4; OS: Windows 6.19; UI render: default; 
TinderBox: Win-x86@42, Branch:master, Time: 2017-08-02_23:46:51
Locale: nl-NL (nl_NL); Calc: CL

but not in
Version: 5.4.0.3
Build ID: 7556cbc6811c9d992f4064ab9287069087d7f62c
CPU threads: 4; OS: Windows 6.2; UI render: default; 
Locale: nl-NL (nl_NL); Calc: CL


User-Agent: Mozilla/5.0 (Windows NT 6.2; WOW64; rv:52.0) Gecko/20100101 Firefox/52.0
Comment 1 Telesto 2017-08-04 18:03:00 UTC 
Created attachment 135143 [details]
Example file
Comment 2 raal 2017-08-05 13:42:11 UTC 
I can confirm with Version: 6.0.0.0.alpha0+
Build ID: b032f21b5ea0011f2b134d758b696604c18dfa16
CPU threads: 1; OS: Windows 6.1; UI render: default; 
TinderBox: Win-x86@42, Branch:master, Time: 2017-08-04_23:35:39

regression
Comment 3 raal 2017-08-05 15:12:07 UTC 
This seems to have begun at the below commit.
Adding Cc: to Dennis Francis ; Could you possibly take a look at this one?
Thanks

author	Dennis Francis <dennis.francis@collabora.co.uk>	2017-07-21 07:42:20 (GMT)
committer	Markus Mohrhard <markus.mohrhard@googlemail.com>	2017-07-27 19:30:54 (GMT)
commit 04461743d75f8cffb5906ab52d772089c44a7780 (patch)
tree 24032c105b1d71a75abe38ee7e822ce2b1a09063
parent e89548c16b39cd716925cc23f2eb9e54748233c7 (diff)
tdf#108887 : Move the code of ScDrawView::CheckOle()...
Comment 4 Dennis Francis 2017-08-07 05:55:44 UTC 
This is fixed by Caolan's patch @ https://cgit.freedesktop.org/libreoffice/core/commit/?id=30f384b5bd2d8ec7f50ad0c85c43611313044f8c in master. Please verify.

I have submitted this patch for 5-4 branch at https://gerrit.libreoffice.org/40818
Comment 5 Dennis Francis 2017-08-07 07:56:39 UTC 
Also backported to 5-4