Bug 132368 - FILESAVE PPTX: text changes are reproducibly lost (SmartArt)
Summary: FILESAVE PPTX: text changes are reproducibly lost (SmartArt)
Status: RESOLVED FIXED
Alias: None
Product: LibreOffice
Classification: Unclassified
Component: Impress (show other bugs)
Version:
(earliest affected)
6.3.0.4 release
Hardware: All All
: medium normal
Assignee: Miklos Vajna
URL:
Whiteboard: target:7.2.0 target:7.1.2 target:7.0.6
Keywords: bibisected, bisected, filter:ooxml, regression
Depends on:
Blocks: OOXML-SmartArt
  Show dependency treegraph
 
Reported: 2020-04-23 23:07 UTC by Gerald Pfeifer
Modified: 2024-08-18 10:52 UTC (History)
3 users (show)

See Also:
Crash report or crash signature:


Attachments
Sample PPTX document (52.28 KB, application/vnd.openxmlformats-officedocument.presentationml.presentation)
2020-04-23 23:08 UTC, Gerald Pfeifer
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Gerald Pfeifer 2020-04-23 23:07:49 UTC
Description:
Changes made to text elements in a specific SmartArt are lost upon saving as PPTX.

Steps to Reproduce:
1. Open sample PPTX document
2. Edit text in the top two / left two elements
3. Save document (still as PPTX)
4. Restart LibreOffice
5. Open saved document

Actual Results:
All text changes have disappeared, reverted to the original status.

Expected Results:
Text changes survive safe - restart - open cycle.


Reproducible: Always


User Profile Reset: No



Additional Info:
Version: 7.0.0.0.alpha0+
Build ID: 2e9261415006ce782a7ae699c69f5be3cec29dff
CPU threads: 4; OS: Linux 5.6; UI render: default; VCL: gtk3; 
TinderBox: Linux-rpm_deb-x86_64@86-TDF, Branch:master, Time: 2020-04-22_02:33:36
Locale: en-US (en_US.UTF-8); UI-Language: en-US
Calc: threaded
Comment 1 Gerald Pfeifer 2020-04-23 23:08:43 UTC
Created attachment 159880 [details]
Sample PPTX document
Comment 2 Aron Budea 2020-04-26 06:23:43 UTC
Confirmed in LO 7.0.0.0.alpha0+ (f924658e45f256544e43c3fdb2af9b585d0f0933) & 6.3.0.4 / Ubuntu.
Edited text is retained after save&reload in 6.2.0.3.
=> regression
Comment 3 Aron Budea 2020-04-27 04:21:09 UTC
Bibisected to the following commit using repo bibisect-linux-64-6.3.

https://cgit.freedesktop.org/libreoffice/core/commit/?id=aafaf1f55fa413ad49d4556cf7c0a713dd206ae4
author		Grzegorz Araminowicz <grzegorz.araminowicz@collabora.com>	2019-03-13 10:49:30 +0100
committer	Miklos Vajna <vmiklos@collabora.com>	2019-04-08 10:07:35 +0200

PPTX export: save SmartArt as diagram instead of group of shapes
Comment 4 Gerald Pfeifer 2020-08-06 23:12:30 UTC
Hej Miklos, I figured you might be interested in this regression since
you committed the patch (as part of last year's SmartArt work).
Comment 5 Miklos Vajna 2021-02-23 13:03:57 UTC
I'll take a look.
Comment 6 Commit Notification 2021-02-25 23:31:14 UTC
Miklos Vajna committed a patch related to this issue.
It has been pushed to "master":

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

tdf#132368 svx: empty the interop grab-bag on ending text edit

It will be available in 7.2.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 7 Miklos Vajna 2021-02-26 08:15:46 UTC
I think this is now fixed on master.
Comment 8 Xisco Faulí 2021-02-26 08:57:26 UTC
I do confirm the issue is fixed in

Version: 7.2.0.0.alpha0+ / LibreOffice Community
Build ID: 43e306ebe61ef5a849dcfd1e74b697be3b3fe716
CPU threads: 4; OS: Linux 5.7; UI render: default; VCL: gtk3
Locale: en-US (en_US.UTF-8); UI: en-US
Calc: threaded

@Miklos, thanks for fixing this issue!!
Comment 9 Timur 2021-02-26 14:27:16 UTC Comment hidden (obsolete)
Comment 10 Gerald Pfeifer 2021-02-26 14:47:01 UTC Comment hidden (obsolete)
Comment 11 Timur 2021-02-26 18:48:24 UTC
(In reply to Gerald Pfeifer from comment #10)
> (In reply to Timur from comment #9)
> > attachment 159880 [details] seems to suffer from fileopen bug for shape text
> > position, since 4.2.x. Previously,text size was too small but later text
> > position is not fully correct  and in center of shape like in MSO. 
> > As always, needs to be searched first and See Also to this bug.
> 
> Timur, do you mean bug #132302 ?

Yes. You should always link via See Also, that you did only now. 
Also, once file is uploaded no need to upload again, just write attachment 12345.
Comment 12 Commit Notification 2021-03-01 11:52:46 UTC
Miklos Vajna committed a patch related to this issue.
It has been pushed to "libreoffice-7-1":

https://git.libreoffice.org/core/commit/4bf14305dfa22a3e4084b630b6924a718d7da3ce

tdf#132368 svx: empty the interop grab-bag on ending text edit

It will be available in 7.1.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 Commit Notification 2021-03-03 09:34:00 UTC
Miklos Vajna committed a patch related to this issue.
It has been pushed to "libreoffice-7-0":

https://git.libreoffice.org/core/commit/4ddde32ea6807eaa686ad25aa18ea24a8b834df8

tdf#132368 svx: empty the interop grab-bag on ending text edit

It will be available in 7.0.6.

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.