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

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