[Libreoffice-commits] core.git: Branch 'libreoffice-5-3' - xmloff/source

Michael Stahl mstahl at redhat.com
Tue May 30 15:59:26 UTC 2017


 xmloff/source/draw/shapeexport.cxx |   15 ++++++++++-----
 1 file changed, 10 insertions(+), 5 deletions(-)

New commits:
commit 024fa6b3915fd84586b348b5033dcc4312cd8926
Author: Michael Stahl <mstahl at redhat.com>
Date:   Tue May 30 13:11:01 2017 +0200

    tdf#108231 tdf#108109 xmloff: SvxUnoDrawingModel doesn't implement...
    
    ... XStyleFamiliesSupplier, so try to do without.  This is called from
    sw in case SWTRANSFER_OBJECTTYPE_DRAWMODEL.
    
    (regression from 9835a5823e0f559aabbc0e15ea126c82229c4bc7)
    
    Change-Id: Ic99ad4bf0c8e6e24fc10da173ec480c9d29c0363
    (cherry picked from commit db38e3f201517acf22c1773ae9fc4cb2514788b1)
    Reviewed-on: https://gerrit.libreoffice.org/38214
    Tested-by: Jenkins <ci at libreoffice.org>
    Reviewed-by: Miklos Vajna <vmiklos at collabora.co.uk>

diff --git a/xmloff/source/draw/shapeexport.cxx b/xmloff/source/draw/shapeexport.cxx
index f3880cb1c5ab..95013a03a27d 100644
--- a/xmloff/source/draw/shapeexport.cxx
+++ b/xmloff/source/draw/shapeexport.cxx
@@ -385,11 +385,16 @@ void XMLShapeExport::collectShapeAutoStyles(const uno::Reference< drawing::XShap
             // Shapes with a Writer TextBox always have a parent style.
             // If there would be none, then just assign the first available.
             uno::Reference<style::XStyleFamiliesSupplier> xStyleFamiliesSupplier(GetExport().GetModel(), uno::UNO_QUERY);
-            uno::Reference<container::XNameAccess> xStyleFamilies = xStyleFamiliesSupplier->getStyleFamilies();
-            uno::Reference<container::XNameAccess> xFrameStyles = xStyleFamilies->getByName("FrameStyles").get< uno::Reference<container::XNameAccess> >();
-            uno::Sequence<OUString> aFrameStyles = xFrameStyles->getElementNames();
-            if (aFrameStyles.hasElements())
-                aParentName = aFrameStyles[0];
+            if (xStyleFamiliesSupplier.is()) // tdf#108231
+            {
+                uno::Reference<container::XNameAccess> xStyleFamilies = xStyleFamiliesSupplier->getStyleFamilies();
+                uno::Reference<container::XNameAccess> xFrameStyles = xStyleFamilies->getByName("FrameStyles").get< uno::Reference<container::XNameAccess> >();
+                uno::Sequence<OUString> aFrameStyles = xFrameStyles->getElementNames();
+                if (aFrameStyles.hasElements())
+                {
+                    aParentName = aFrameStyles[0];
+                }
+            }
         }
 
         // filter propset


More information about the Libreoffice-commits mailing list