Description: In LO there is some grabbag mechanism which saves MSO properties imported from PPTX file and if nothing changes in the document it can be saved again into the PPTX file. The problem is that if I change the shadow settings, it does not override the imported value and so after resave property changes are not preserved. Steps to Reproduce: 1. Open attached PPTX file exported by MSO 2. Import into Impress 3. Change shadow properties (color, transparency, distance, angle) 4. Save to a new PPTX 5. Import the saved PPTX in LO or in MSO Actual Results: The changes made in Impress are not saved. Expected Results: Changes made in Impress should have effect on the saved PPTX file Reproducible: Always User Profile Reset: No Additional Info: User-Agent: Mozilla/5.0 (Windows NT 6.3; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/60.0.3112.90 Safari/537.36
Created attachment 135536 [details] PPTX file exported from MSO, import this file first while testing
Created attachment 135537 [details] PPTX file exported by LO after changes was made Same attributes are there and same value as in the first attached document. Only the order is different, as LO writes out the properties in this order.
Confirmed in Version: 6.0.0.0.alpha0+ Build ID: ef2e6260fad38e26c8591ea88ded348db618270d CPU threads: 4; OS: Linux 4.10; UI render: default; VCL: gtk3; Locale: ca-ES (ca_ES.UTF-8); Calc: group
Let's turn it into a easy hack. The code is here: oox/source/export/drawingml.cxx Check DrawingML::WriteShapeEffect(...) method for export and oox/source/drawingml/effectpropertiescontext.cxx for import part
Created attachment 138598 [details] An other test case created in MSO
martinb214 committed a patch related to this issue. It has been pushed to "master": http://cgit.freedesktop.org/libreoffice/core/commit/?id=7f42b0f96a2798ae99aa65b84b0db3b2af2b282b tdf#111790: Shadow imported from a PPTX file is not overriden It will be available in 6.1.0. The patch should be included in the daily builds available at http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: http://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
This bug was created as an easy hack and the easy part is fixed now, which makes the second test case (attachment 138598 [details]) working while the first one (attachment 135536 [details]) still shows an issue. Fixing the first test case would be more complicated as an easy hack, so better to close this bug with after the partail fix.