Bug 65718 - Charts imported without blip fills
Summary: Charts imported without blip fills
Status: RESOLVED FIXED
Alias: None
Product: POI
Classification: Unclassified
Component: XSLF (show other bugs)
Version: 5.0.x-dev
Hardware: PC All
: P2 normal (vote)
Target Milestone: ---
Assignee: POI Developers List
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2021-12-02 09:00 UTC by Max
Modified: 2021-12-22 15:38 UTC (History)
1 user (show)


Attachments
Import chart blip fill (868.14 KB, patch)
2021-12-02 09:00 UTC, Max 		Details | Diff
chart-picture-bg.pptx (636.38 KB, application/vnd.openxmlformats-officedocument.presentationml.presentation)
2021-12-02 09:52 UTC, Max 		Details
chart-texture-bg.pptx (54.42 KB, application/vnd.openxmlformats-officedocument.presentationml.presentation)
2021-12-02 09:53 UTC, Max 		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 Max 2021-12-02 09:00:33 UTC 
Created attachment 38112 [details]
Import chart blip fill

The chart blip fill files are not being copied while importing charts.
Suggesting a patch.
Comment 1 PJ Fanning 2021-12-02 09:42:52 UTC 
thanks for the patch - but it breaks an existing test so I can not apply it

Can you have a look at this test?

TestXSLFSheet.testImportContent()
Comment 2 PJ Fanning 2021-12-02 09:48:18 UTC 
Could you attach chart-picture-bg.pptx separately? It is appearing as empty when I apply the svn patch - and this causes the test failure above.
Comment 3 Max 2021-12-02 09:52:11 UTC 
Created attachment 38113 [details]
chart-picture-bg.pptx
Comment 4 Max 2021-12-02 09:53:00 UTC 
Created attachment 38114 [details]
chart-texture-bg.pptx

Attached both the test files.
Comment 5 Max 2021-12-02 09:55:46 UTC 
Not sure what is the issue with the patch, works for me applying from 
IntelliJ IDEA.
Comment 6 PJ Fanning 2021-12-02 10:10:38 UTC 
applied change with r1895487