sd/qa/unit/data/pptx/tdf152186.pptx      |binary
 sd/qa/unit/import-tests2.cxx             |   17 +++++++++++++++++
 sd/source/filter/eppt/pptx-epptooxml.cxx |   16 ++--------------
 3 files changed, 19 insertions(+), 14 deletions(-)

New commits:
commit bfe33d42a2381bce194cc6fe15b830902f7c8443
Author:     Tibor Nagy <nagy.tib...@nisz.hu>
AuthorDate: Thu Nov 24 14:51:55 2022 +0100
Commit:     László Németh <nem...@numbertext.org>
CommitDate: Tue Nov 29 18:15:11 2022 +0100

    tdf#152186 PPTX export: fix shadow of the shape in theme
    
    Placeholder line styles defined extra effectStyle, resulting
    bad export with shadows.
    
    Follow-up to commit 25e9766e9b87342b6f1b29dc146d5f33e12157a7
    (tdf#92222 PPTX export: fix line width of the shape in theme)
    
    Change-Id: I02f1edea84b50dbb9b0e0a59b6062a29894afde4
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/143233
    Tested-by: László Németh <nem...@numbertext.org>
    Reviewed-by: László Németh <nem...@numbertext.org>

diff --git a/sd/qa/unit/data/pptx/tdf152186.pptx 
b/sd/qa/unit/data/pptx/tdf152186.pptx
new file mode 100644
index 000000000000..c4160f2091a4
Binary files /dev/null and b/sd/qa/unit/data/pptx/tdf152186.pptx differ
diff --git a/sd/qa/unit/import-tests2.cxx b/sd/qa/unit/import-tests2.cxx
index 43ea5583a579..46d1966a2e1a 100644
--- a/sd/qa/unit/import-tests2.cxx
+++ b/sd/qa/unit/import-tests2.cxx
@@ -68,6 +68,7 @@ public:
     {
     }
 
+    void testTdf152186();
     void testTdf93868();
     void testTdf95932();
     void testTdf99030();
@@ -147,6 +148,7 @@ public:
 
     CPPUNIT_TEST_SUITE(SdImportTest2);
 
+    CPPUNIT_TEST(testTdf152186);
     CPPUNIT_TEST(testTdf93868);
     CPPUNIT_TEST(testTdf95932);
     CPPUNIT_TEST(testTdf99030);
@@ -225,6 +227,21 @@ public:
     CPPUNIT_TEST_SUITE_END();
 };
 
+void SdImportTest2::testTdf152186()
+{
+    loadFromURL(u"pptx/tdf152186.pptx");
+    saveAndReload("Impress MS PowerPoint 2007 XML");
+
+    bool bHasShadow;
+    const SdrPage* pPage = GetPage(1);
+    for (size_t i = 0; i < pPage->GetObjCount(); ++i)
+    {
+        uno::Reference<beans::XPropertySet> xShape(getShapeFromPage(i, 0));
+        xShape->getPropertyValue("Shadow") >>= bHasShadow;
+        CPPUNIT_ASSERT(!bHasShadow);
+    }
+}
+
 void SdImportTest2::testTdf93868()
 {
     createSdImpressDoc("pptx/tdf93868.pptx");
diff --git a/sd/source/filter/eppt/pptx-epptooxml.cxx 
b/sd/source/filter/eppt/pptx-epptooxml.cxx
index a9edd475aeff..842c7944d0c3 100644
--- a/sd/source/filter/eppt/pptx-epptooxml.cxx
+++ b/sd/source/filter/eppt/pptx-epptooxml.cxx
@@ -2035,22 +2035,10 @@ ShapeExport& 
PowerPointShapeExport::WritePlaceholderReferenceTextBody(
       </a:lnStyleLst>\
       <a:effectStyleLst>\
         <a:effectStyle>\
-          <a:effectLst>\
-            <a:outerShdw blurRad=\"40000\" dist=\"20000\" dir=\"5400000\" 
rotWithShape=\"0\">\
-              <a:srgbClr val=\"000000\">\
-                <a:alpha val=\"38000\"/>\
-              </a:srgbClr>\
-            </a:outerShdw>\
-          </a:effectLst>\
+          <a:effectLst/>\
         </a:effectStyle>\
         <a:effectStyle>\
-          <a:effectLst>\
-            <a:outerShdw blurRad=\"40000\" dist=\"23000\" dir=\"5400000\" 
rotWithShape=\"0\">\
-              <a:srgbClr val=\"000000\">\
-                <a:alpha val=\"35000\"/>\
-              </a:srgbClr>\
-            </a:outerShdw>\
-          </a:effectLst>\
+          <a:effectLst/>\
         </a:effectStyle>\
         <a:effectStyle>\
           <a:effectLst>\

Reply via email to