tdf#159258 sd: fix to show objects in slideshow if they have

fillstyle or linestyle.

Also the shape will be appeared in print and pdf view.
(Powerpoint doing the same.)

TODO: the placeholder bitmap and the default text
was not removed from the slideshow/print/pdf view.

Change-Id: Ifadc9a692d77b60a7e3514afe8e6ea5cab0018c9
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/164163
Tested-by: Jenkins
Reviewed-by: Balazs Varga <balazs.varga.extern@allotropia.de>

1 files changed, 19 insertions, 1 deletions

diff --git a/slideshow/source/engine/shapes/shapeimporter.cxx b/slideshow/source/engine/shapes/shapeimporter.cxx
index 92162eeb60ad..7823a5588efd 100644
--- a/slideshow/source/engine/shapes/shapeimporter.cxx
+++ b/slideshow/source/engine/shapes/shapeimporter.cxx
@@ -346,7 +346,25 @@ bool ShapeImporter::isSkip(
                           "IsEmptyPresentationObject") &&
         bEmpty )
     {
-        return true;
+        // check object have fill or linestyle, if have, it should be visible
+        drawing::FillStyle aFillStyle{ drawing::FillStyle_NONE };
+        if (getPropertyValue(aFillStyle,
+            xPropSet, "FillStyle") &&
+            aFillStyle != drawing::FillStyle_NONE)
+        {
+            bEmpty = false;
+        }
+
+        drawing::LineStyle aLineStyle{ drawing::LineStyle_NONE };
+        if (bEmpty && getPropertyValue(aLineStyle,
+            xPropSet, "LineStyle") &&
+            aLineStyle != drawing::LineStyle_NONE)
+        {
+            bEmpty = false;
+        }
+
+        if (bEmpty)
+            return true;
     }
 
     //skip shapes which corresponds to annotations