Summary: | Removing last slide then appending new slide leads to corrupted PowerPoint file | ||
---|---|---|---|
Product: | POI | Reporter: | François Beaune <dictoon> |
Component: | XSLF | Assignee: | POI Developers List <dev> |
Status: | RESOLVED FIXED | ||
Severity: | normal | ||
Priority: | P2 | ||
Version: | 3.16-FINAL | ||
Target Milestone: | --- | ||
Hardware: | PC | ||
OS: | All | ||
Attachments: |
Minimal Java repro case
Input file for the repro case Output of the repro case |
Created attachment 35319 [details]
Input file for the repro case
This file is simply an unmodified blank document with a single slide, saved with PowerPoint 2013.
Created attachment 35320 [details]
Output of the repro case
This file is reported as corrupted by PowerPoint.
I just pushed a fix for this as pull request #70: https://github.com/apache/poi/pull/70 https://svn.apache.org/viewvc?view=revision&revision=1808294 has been merged Awesome, thank you very much! Will the fix be part of the next release, which I presume is 3.17? 3.17 was released before this change was added. I was under the impression that 3.17 final hadn't been released yet and that you would maybe fold this fix into the current or next beta. That said, it's of course fine if the fix is in 3.18. |
Created attachment 35318 [details] Minimal Java repro case Please find attached a minimal repro case that: 1. Loads a basic PowerPoint file with a single slide. 2. Removes that slides. 3. Appends a new slide. 4. Save the resulting document. The document is reported as corrupted by PowerPoint 2013 and PowerPoint 2016 which propose to fix it.