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: NEW
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: 2018-11-08 07:03 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

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