[Bug 61515] New: Removing last slide then appending new slide leads to corrupted PowerPoint file

classic Classic list List threaded Threaded
8 messages Options
Reply | Threaded
Open this post in threaded view
|

[Bug 61515] New: Removing last slide then appending new slide leads to corrupted PowerPoint file

Bugzilla from bugzilla@apache.org
https://bz.apache.org/bugzilla/show_bug.cgi?id=61515

            Bug ID: 61515
           Summary: Removing last slide then appending new slide leads to
                    corrupted PowerPoint file
           Product: POI
           Version: 3.16-FINAL
          Hardware: PC
            Status: NEW
          Severity: normal
          Priority: P2
         Component: XSLF
          Assignee: [hidden email]
          Reporter: [hidden email]
  Target Milestone: ---

Created attachment 35318
  --> https://bz.apache.org/bugzilla/attachment.cgi?id=35318&action=edit
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.

--
You are receiving this mail because:
You are the assignee for the bug.
---------------------------------------------------------------------
To unsubscribe, e-mail: [hidden email]
For additional commands, e-mail: [hidden email]

Reply | Threaded
Open this post in threaded view
|

[Bug 61515] Removing last slide then appending new slide leads to corrupted PowerPoint file

Bugzilla from bugzilla@apache.org
https://bz.apache.org/bugzilla/show_bug.cgi?id=61515

--- Comment #1 from François Beaune <[hidden email]> ---
Created attachment 35319
  --> https://bz.apache.org/bugzilla/attachment.cgi?id=35319&action=edit
Input file for the repro case

This file is simply an unmodified blank document with a single slide, saved
with PowerPoint 2013.

--
You are receiving this mail because:
You are the assignee for the bug.
---------------------------------------------------------------------
To unsubscribe, e-mail: [hidden email]
For additional commands, e-mail: [hidden email]

Reply | Threaded
Open this post in threaded view
|

[Bug 61515] Removing last slide then appending new slide leads to corrupted PowerPoint file

Bugzilla from bugzilla@apache.org
In reply to this post by Bugzilla from bugzilla@apache.org
https://bz.apache.org/bugzilla/show_bug.cgi?id=61515

--- Comment #2 from François Beaune <[hidden email]> ---
Created attachment 35320
  --> https://bz.apache.org/bugzilla/attachment.cgi?id=35320&action=edit
Output of the repro case

This file is reported as corrupted by PowerPoint.

--
You are receiving this mail because:
You are the assignee for the bug.
---------------------------------------------------------------------
To unsubscribe, e-mail: [hidden email]
For additional commands, e-mail: [hidden email]

Reply | Threaded
Open this post in threaded view
|

[Bug 61515] Removing last slide then appending new slide leads to corrupted PowerPoint file

Bugzilla from bugzilla@apache.org
In reply to this post by Bugzilla from bugzilla@apache.org
https://bz.apache.org/bugzilla/show_bug.cgi?id=61515

Alain Fagot Bearez <[hidden email]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 OS|                            |All

--- Comment #3 from Alain Fagot Bearez <[hidden email]> ---
I just pushed a fix for this as pull request #70:

https://github.com/apache/poi/pull/70

--
You are receiving this mail because:
You are the assignee for the bug.
---------------------------------------------------------------------
To unsubscribe, e-mail: [hidden email]
For additional commands, e-mail: [hidden email]

Reply | Threaded
Open this post in threaded view
|

[Bug 61515] Removing last slide then appending new slide leads to corrupted PowerPoint file

Bugzilla from bugzilla@apache.org
In reply to this post by Bugzilla from bugzilla@apache.org
https://bz.apache.org/bugzilla/show_bug.cgi?id=61515

PJ Fanning <[hidden email]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
         Resolution|---                         |FIXED

--- Comment #4 from PJ Fanning <[hidden email]> ---
https://svn.apache.org/viewvc?view=revision&revision=1808294 has been merged

--
You are receiving this mail because:
You are the assignee for the bug.
---------------------------------------------------------------------
To unsubscribe, e-mail: [hidden email]
For additional commands, e-mail: [hidden email]

Reply | Threaded
Open this post in threaded view
|

[Bug 61515] Removing last slide then appending new slide leads to corrupted PowerPoint file

Bugzilla from bugzilla@apache.org
In reply to this post by Bugzilla from bugzilla@apache.org
https://bz.apache.org/bugzilla/show_bug.cgi?id=61515

--- Comment #5 from François Beaune <[hidden email]> ---
Awesome, thank you very much!

Will the fix be part of the next release, which I presume is 3.17?

--
You are receiving this mail because:
You are the assignee for the bug.
---------------------------------------------------------------------
To unsubscribe, e-mail: [hidden email]
For additional commands, e-mail: [hidden email]

Reply | Threaded
Open this post in threaded view
|

[Bug 61515] Removing last slide then appending new slide leads to corrupted PowerPoint file

Bugzilla from bugzilla@apache.org
In reply to this post by Bugzilla from bugzilla@apache.org
https://bz.apache.org/bugzilla/show_bug.cgi?id=61515

--- Comment #6 from PJ Fanning <[hidden email]> ---
3.17 was released before this change was added.

--
You are receiving this mail because:
You are the assignee for the bug.
---------------------------------------------------------------------
To unsubscribe, e-mail: [hidden email]
For additional commands, e-mail: [hidden email]

Reply | Threaded
Open this post in threaded view
|

[Bug 61515] Removing last slide then appending new slide leads to corrupted PowerPoint file

Bugzilla from bugzilla@apache.org
In reply to this post by Bugzilla from bugzilla@apache.org
https://bz.apache.org/bugzilla/show_bug.cgi?id=61515

--- Comment #7 from François Beaune <[hidden email]> ---
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.

--
You are receiving this mail because:
You are the assignee for the bug.
---------------------------------------------------------------------
To unsubscribe, e-mail: [hidden email]
For additional commands, e-mail: [hidden email]