Bug 105695 - FILEOPEN PPTX: wrong background color in PPTX created with Evento template
Summary: FILEOPEN PPTX: wrong background color in PPTX created with Evento template
Status: RESOLVED FIXED
Alias: None
Product: LibreOffice
Classification: Unclassified
Component: Impress (show other bugs)
Version:
(earliest affected)
5.2.0.0.alpha1
Hardware: All All
: medium normal
Assignee: Not Assigned
URL:
Whiteboard:
Keywords: bibisected, bisected, filter:pptx, regression
Depends on:
Blocks: PPTX
  Show dependency treegraph
 
Reported: 2017-02-02 11:30 UTC by Ofir
Modified: 2022-07-01 09:21 UTC (History)
5 users (show)

See Also:
Crash report or crash signature:


Attachments
PPTX created with Evento template showing the wrong background (1.08 MB, application/vnd.openxmlformats-officedocument.presentationml.presentation)
2017-02-02 11:30 UTC, Ofir
Details
PDF exported with PowerPoint 2013 showing expected result (473.41 KB, application/pdf)
2017-02-02 11:31 UTC, Ofir
Details
simplifier sample file (244.66 KB, application/vnd.openxmlformats-officedocument.presentationml.presentation)
2019-10-03 15:40 UTC, Xisco Faulí
Details
another sample with the shape in the master document (696.14 KB, application/vnd.openxmlformats-officedocument.presentationml.presentation)
2019-10-04 08:39 UTC, Xisco Faulí
Details
very simple sample (33.24 KB, application/vnd.openxmlformats-officedocument.presentationml.presentation)
2019-10-04 09:15 UTC, Xisco Faulí
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Ofir 2017-02-02 11:30:49 UTC
Created attachment 130845 [details]
PPTX created with Evento template showing the wrong background

The attached PPTX was created with the Evento template in PowerPoint 2013.
The large shape in the Master Slide has "Slide background fill" which is gray.

In LO 5.2.5 and 5.3 the shape has red background.
In LO 5.1.4.2 it was much better so this is a regression.
Comment 1 Ofir 2017-02-02 11:31:17 UTC
Created attachment 130846 [details]
PDF exported with PowerPoint 2013 showing expected result
Comment 2 Xisco Faulí 2017-02-02 13:23:04 UTC
Confirmed in

Version: 5.4.0.0.alpha0+
Build ID: fc53cce64400430cdc21f79c959d75fb9a26d13d
CPU Threads: 4; OS Version: Linux 4.8; UI Render: default; VCL: gtk2; 
Locale: ca-ES (ca_ES.UTF-8); Calc: group

and 

Version: 5.2.0.0.alpha1+
Build ID: 5b168b3fa568e48e795234dc5fa454bf24c9805e
CPU Threads: 4; OS Version: Linux 4.8; UI Render: default; 
Locale: ca-ES (ca_ES.UTF-8)

but not in

Version: 5.0.0.0.alpha1+
Build ID: 0db96caf0fcce09b87621c11b584a6d81cc7df86
Locale: ca-ES (ca_ES.UTF-8)
Comment 3 Xisco Faulí 2017-02-02 13:53:05 UTC
Regression introduced by:

author	Matus Uzak <matus.uzak@gmail.com>	2016-03-08 20:44:21 (GMT)
committer	Katarina Behrens <Katarina.Behrens@cib.de>	2016-03-23 06:49:01 (GMT)
commit f3d1ac75c4b7fa63022e54a9cbff46ba99535076 (patch)
tree d327b5f81c4eaf7869d8349274d308ed497f19f7
parent 15b53976e5d119877e53f34b34cee33a5f2883fd (diff)

tdf#93868: PPTX import: Incorrect inheritance of shape fill properties
DrawingML: The useBgFill attribute specifies that the shape fill
should be set to that of the slide background.

Adding Cc: to Matus Uzak

the issue is fixed if we delete 'if (!pBackgroundPropertiesPtr)' in oox/source/ppt/pptshapegroupcontext.cxx but the unittest fails.
Comment 4 Xisco Faulí 2017-02-02 13:56:38 UTC
Miklos Vajna has done some recent changes in that area too: 59339dec1ce56213dc74a06af2f0d35ac1c534d7. Adding him, in case he want to take a look as well.
Comment 5 Xisco Faulí 2017-02-05 20:17:08 UTC
Hi Tamás,
I'm adding you as I thought you might be interested in this one...
Comment 6 Xisco Faulí 2017-09-11 08:36:11 UTC
Dear developer,
This bug has been in ASSIGNED status for more than 3 months without any
activity. Resetting it to NEW.
Please assigned it back to yourself if you're still working on this.
Comment 7 Ofir 2017-11-09 08:35:32 UTC
Still reproducible:

Version: 6.0.0.0.alpha1
Build ID: c1d1f859b268f650143d48f294999cda0fa57350
CPU threads: 1; OS: Linux 4.10; UI render: default; VCL: gtk2; 
Locale: en-US (en_US.UTF-8); Calc: group
Comment 8 Ofir 2018-09-28 03:32:58 UTC
Reproducible with LO 6.1.0.3.
Comment 9 Lars Jødal 2018-11-08 07:03:16 UTC
Still reproducible with 6.2.0.0.alpha1

Version: 6.2.0.0.alpha1 (x64)
Build ID: ff46ad24d1d3cbcea45895520483ed1fd4ff488b
CPU tråde: 4; Styresystem: Windows 10.0; Gengiver af brugergrænseflade: GL; VCL: win; 
Lokalisering: da-DK (da_DK); Calc: CL
Comment 10 Ofir 2019-05-24 21:35:24 UTC
Still reproducible in:

Version: 6.3.0.0.alpha1
Build ID: 547edd20e527fb02900f6174973770d26306e2e7
CPU threads: 1; OS: Linux 4.18; UI render: default; VCL: gtk3; 
Locale: en-US (en_IL); UI-Language: en-US
Comment 11 Xisco Faulí 2019-10-03 15:40:08 UTC
Created attachment 154738 [details]
simplifier sample file

The problem is in the shape using <p:sp useBgFill="1">
Comment 12 Xisco Faulí 2019-10-03 15:59:57 UTC
Similar to bug 123684
Comment 13 Xisco Faulí 2019-10-04 08:20:55 UTC Comment hidden (obsolete)
Comment 14 Xisco Faulí 2019-10-04 08:39:29 UTC
Created attachment 154750 [details]
another sample with the shape in the master document
Comment 15 Xisco Faulí 2019-10-04 09:14:30 UTC
How to reproduce it from scratch:
1. Open MSO Powerpoint
2. Insert a shape -> shape is blue ( at least for me )
3. Change shapes' filling to 'slide background fill'
4. Save
5. Open in LibreOffice

-> The shape is blue
Comment 16 Xisco Faulí 2019-10-04 09:15:59 UTC Comment hidden (obsolete)
Comment 17 Xisco Faulí 2019-10-04 09:19:36 UTC Comment hidden (obsolete)
Comment 18 Xisco Faulí 2019-10-04 12:56:54 UTC
(In reply to Xisco Faulí from comment #15)
> How to reproduce it from scratch:
> 1. Open MSO Powerpoint
> 2. Insert a shape -> shape is blue ( at least for me )
> 3. Change shapes' filling to 'slide background fill'
> 4. Save
> 5. Open in LibreOffice
> 
> -> The shape is blue

Reproduced in bug 127964
Comment 19 Xisco Faulí 2019-10-04 13:01:35 UTC
Same problem if the shape is in the master slide