tdf#127791 write events at the end of groups instead of the start

because the group shape import applies the events over the children it has when
it reads the events, so putting it before exportShapes means it get applied
over 0 children

Change-Id: If7d09865a7a34d8d13440eb559ec97580792bd4e
Reviewed-on: https://gerrit.libreoffice.org/82160
Tested-by: Jenkins
Reviewed-by: Caolán McNamara <caolanm@redhat.com>
Tested-by: Caolán McNamara <caolanm@redhat.com>

1 files changed, 7 insertions, 1 deletions

diff --git a/xmloff/source/draw/shapeexport.cxx b/xmloff/source/draw/shapeexport.cxx
index ddd128d1d8b0..c7640016786c 100644
--- a/xmloff/source/draw/shapeexport.cxx
+++ b/xmloff/source/draw/shapeexport.cxx
@@ -1883,7 +1883,6 @@ void XMLShapeExport::ImpExportGroupShape( const uno::Reference< drawing::XShape
         SvXMLElementExport aPGR(mrExport, XML_NAMESPACE_DRAW, XML_G, bCreateNewline, true);
 
         ImpExportDescription( xShape ); // #i68101#
-        ImpExportEvents( xShape );
         ImpExportGluePoints( xShape );
 
         // #89764# if export of position is suppressed for group shape,
@@ -1900,6 +1899,13 @@ void XMLShapeExport::ImpExportGroupShape( const uno::Reference< drawing::XShape
 
         // write members
         exportShapes( xShapes, nFeatures, pRefPoint );
+
+        /* tdf#127791 write events after shapes are exported because
+           the group shape import applies the events over the
+           children it has when it reads the events, so putting
+           it before exportShapes means it get applied over 0 children
+        */
+        ImpExportEvents( xShape );
     }
 }