Bug 65718 - Charts imported without blip fills
Summary: Charts imported without blip fills
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
Depends on:
Reported: 2021-12-02 09:00 UTC by Max
Modified: 2021-12-22 15:38 UTC (History)
1 user (show)

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
chart-texture-bg.pptx (54.42 KB, application/vnd.openxmlformats-officedocument.presentationml.presentation)
2021-12-02 09:53 UTC, Max

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?

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]
Comment 4 Max 2021-12-02 09:53:00 UTC
Created attachment 38114 [details]

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