Bug 135342 - Impress generates a wrong PPTX file after Minimize presentation
Summary: Impress generates a wrong PPTX file after Minimize presentation
Status: VERIFIED FIXED
Alias: None
Product: LibreOffice
Classification: Unclassified
Component: Impress (show other bugs)
Version:
(earliest affected)
4.4 all versions
Hardware: All All
: medium normal
Assignee: Mike Kaganski
URL:
Whiteboard: target:7.1.0 target:7.0.2 target:6.4.7
Keywords: filter:pptx, regression
Depends on:
Blocks:
 
Reported: 2020-07-31 12:29 UTC by Roman Kuznetsov
Modified: 2020-09-21 21:49 UTC (History)
1 user (show)

See Also:
Crash report or crash signature:


Attachments
Original PPTX file (5.77 MB, application/vnd.openxmlformats-officedocument.presentationml.presentation)
2020-07-31 12:30 UTC, Roman Kuznetsov
Details
Result - minimized PPTX file (155.97 KB, application/vnd.openxmlformats-officedocument.presentationml.presentation)
2020-07-31 12:31 UTC, Roman Kuznetsov
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Roman Kuznetsov 2020-07-31 12:29:51 UTC
Description:
Impress generate wrong PPTX file after Minimize presentation

Steps to Reproduce:
1. Open PPTX from attach (I made it in MSO 2016) in LobreOffice Impress
2. Select menu Tools-Minimize presentation
3. Use default settings in the wizard. On 5 step select "Duplicate presentation before applying changes"
4. Press "Finish" button
5. Select type file PowerPoint 2007-365 (*.pptx) and press "Save"
6. Try open saved file in PowerPoint => it will show an error with "this file is corrupt and con't be open" (or something, I see it on Russian) 

Actual Results:
PowerPoint can't open saved PPTX file

Expected Results:
PowerPoint opens saved PPTX file fine


Reproducible: Always


User Profile Reset: No



Additional Info:
Version: 7.1.0.0.alpha0+ (x64)
Build ID: <buildversion>
CPU threads: 4; OS: Windows 10.0 Build 18362; UI render: default; VCL: win
Locale: ru-RU (ru_RU); UI: en-US
Calc: threaded

I tried it in 6.2 and got the same problem
Comment 1 Roman Kuznetsov 2020-07-31 12:30:50 UTC
Created attachment 163808 [details]
Original PPTX file
Comment 2 Roman Kuznetsov 2020-07-31 12:31:21 UTC
Created attachment 163809 [details]
Result - minimized PPTX file
Comment 3 Roman Kuznetsov 2020-07-31 12:32:32 UTC
But if you save file in the wizard as ODT, open it and resave it as PPTX, then PowerPoint opens it fine!
Comment 4 Xisco Faulí 2020-07-31 16:10:52 UTC
Reproduced in

Version: 7.1.0.0.alpha0+
Build ID: 231e1e416c039d1f9724962a89cf0573a3db48a2
CPU threads: 4; OS: Linux 4.19; UI render: default; VCL: gtk3
Locale: en-US (en_US.UTF-8); UI: en-US
Calc: threaded

and MSO 2010
Comment 5 Xisco Faulí 2020-07-31 16:14:45 UTC
Also reproduced in

Version: 5.2.0.0.alpha0+
Build ID: 3ca42d8d51174010d5e8a32b96e9b4c0b3730a53
Threads 4; Ver: 4.19; Render: default; 

Locale: en-US (en_US.UTF-8)
Comment 6 Xisco Faulí 2020-07-31 16:23:25 UTC
Also reproduced in

Version: 4.3.0.0.alpha1+
Build ID: c15927f20d4727c3b8de68497b6949e72f9e6e9e
Comment 7 Roman Kuznetsov 2020-07-31 20:24:02 UTC
If on step 3 you select "Lossless compression" instead "JPG compression" then you'll get a fine PPTX file that PP can open
Comment 8 Mike Kaganski 2020-07-31 20:59:39 UTC
The file that gets generated as "PPTX" by LO in case of JPEG compression is in fact an ODP in disguise :-)

tdf#121497 looks superficially similar (but given the difference between PNG and JPEG compression, the problem is unlikely in filter chooser... ah, and it's reproduced on Linux!)
Comment 9 Mike Kaganski 2020-08-16 16:07:11 UTC
https://gerrit.libreoffice.org/c/core/+/100821
Comment 10 Mike Kaganski 2020-08-16 16:11:20 UTC
Regression after the fix to i#121857
Comment 11 Commit Notification 2020-08-16 18:08:52 UTC
Mike Kaganski committed a patch related to this issue.
It has been pushed to "master":

https://git.libreoffice.org/core/commit/99c6c89deed41457d35c382fc7d6a4cd5191f7d7

tdf#135342: filter list must include extensions

It will be available in 7.1.0.

The patch should be included in the daily builds available at
https://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More
information about daily builds can be found at:
https://wiki.documentfoundation.org/Testing_Daily_Builds

Affected users are encouraged to test the fix and report feedback.
Comment 12 Commit Notification 2020-08-16 18:53:21 UTC
Mike Kaganski committed a patch related to this issue.
It has been pushed to "libreoffice-7-0":

https://git.libreoffice.org/core/commit/bbd0d424263a21dd0ffdc42d1201f86e65358f25

tdf#135342: filter list must include extensions

It will be available in 7.0.2.

The patch should be included in the daily builds available at
https://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More
information about daily builds can be found at:
https://wiki.documentfoundation.org/Testing_Daily_Builds

Affected users are encouraged to test the fix and report feedback.
Comment 13 Xisco Faulí 2020-08-31 10:21:35 UTC
Verified in

Version: 7.1.0.0.alpha0+
Build ID: e2f4e65a7b8024c00b049eebf0d87637efda7f24
CPU threads: 4; OS: Linux 4.19; UI render: default; VCL: gtk3
Locale: en-US (en_US.UTF-8); UI: en-US
Calc: threaded

@Mike Kaganski, thanks for fixing this issue!!
Comment 14 Commit Notification 2020-09-21 21:49:44 UTC
Mike Kaganski committed a patch related to this issue.
It has been pushed to "libreoffice-6-4":

https://git.libreoffice.org/core/commit/20e557dc730d66453d32f9268381274c268cabd4

tdf#135342: filter list must include extensions

It will be available in 6.4.7.

The patch should be included in the daily builds available at
https://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More
information about daily builds can be found at:
https://wiki.documentfoundation.org/Testing_Daily_Builds

Affected users are encouraged to test the fix and report feedback.