[libreoffice] Resolves: fdo#84043 don't create duplicate Mirrored props

Caolán McNamara caolanm at fedoraproject.org
Thu Nov 27 11:37:24 UTC 2014


commit de4588134bcc111eb0554403495e0d67f9d67f31
Author: Caolán McNamara <caolanm at redhat.com>
Date:   Thu Nov 27 11:37:19 2014 +0000

    Resolves: fdo#84043 don't create duplicate Mirrored props

 ...o-84043-don-t-create-duplicate-Mirrored-X.patch |   78 ++++++++++++++++++++
 libreoffice.spec                                   |    1 +
 2 files changed, 79 insertions(+), 0 deletions(-)
---
diff --git a/0001-Resolves-fdo-84043-don-t-create-duplicate-Mirrored-X.patch b/0001-Resolves-fdo-84043-don-t-create-duplicate-Mirrored-X.patch
new file mode 100644
index 0000000..7943bb7
--- /dev/null
+++ b/0001-Resolves-fdo-84043-don-t-create-duplicate-Mirrored-X.patch
@@ -0,0 +1,78 @@
+From 10924e1e0c3b88f5650994b6943193d23f99109d Mon Sep 17 00:00:00 2001
+From: =?UTF-8?q?Caol=C3=A1n=20McNamara?= <caolanm at redhat.com>
+Date: Thu, 27 Nov 2014 11:30:45 +0000
+Subject: [PATCH] Resolves: fdo#84043 don't create duplicate Mirrored[X|Y]
+ properties
+
+regression from 13ef16423e78d3ea825172594f08c47d2f9bfd09
+
+commit 13ef16423e78d3ea825172594f08c47d2f9bfd09
+Author: Armin Le Grand <alg at apache.org>
+Date:   Wed Nov 21 13:23:01 2012 +0000
+
+    For backward compatibility take mirrorings in setTransformation into account
+
+    Also found an error in SdrObjCustomShape::TRGetBaseGeometry when MirrorY was used
+
+    (cherry picked from commit 4116c33b12d3787c406f0348f89efcb1cf409507)
+
+Change-Id: I7bfb5dea32b8ab8498e3d92975c49b830c81e6fb
+(cherry picked from commit 751e5b32c5c361995bf0ba3295f773341fd92c23)
+---
+ xmloff/source/draw/ximpshap.cxx | 32 ++++++++++++++++++++++++--------
+ 1 file changed, 24 insertions(+), 8 deletions(-)
+
+diff --git a/xmloff/source/draw/ximpshap.cxx b/xmloff/source/draw/ximpshap.cxx
+index 74bb00c..6a8c77a 100644
+--- a/xmloff/source/draw/ximpshap.cxx
++++ b/xmloff/source/draw/ximpshap.cxx
+@@ -3832,22 +3832,38 @@ void SdXMLCustomShapeContext::EndElement()
+ 
+         if(bFlippedX || bFlippedY)
+         {
+-            beans::PropertyValue aNewPoroperty;
++            OUString sName;
+ 
+             if(bFlippedX)
++                sName = "MirroredX";
++            else
++                sName = "MirroredY";
++
++            //fdo#84043 overwrite the property if it already exists, otherwise append it
++            beans::PropertyValue* pItem;
++            std::vector< beans::PropertyValue >::iterator aI(maCustomShapeGeometry.begin());
++            std::vector< beans::PropertyValue >::iterator aE(maCustomShapeGeometry.end());
++            while (aI != aE)
++            {
++                if (aI->Name == sName)
++                    break;
++                ++aI;
++            }
++            if (aI != aE)
+             {
+-                aNewPoroperty.Name = "MirroredX";
++                beans::PropertyValue& rItem = *aI;
++                pItem = &rItem;
+             }
+             else
+             {
+-                aNewPoroperty.Name = "MirroredY";
++                maCustomShapeGeometry.push_back(beans::PropertyValue());
++                pItem = &maCustomShapeGeometry.back();
+             }
+ 
+-            aNewPoroperty.Handle = -1;
+-            aNewPoroperty.Value <<= sal_True;
+-            aNewPoroperty.State = beans::PropertyState_DIRECT_VALUE;
+-
+-            maCustomShapeGeometry.push_back(aNewPoroperty);
++            pItem->Name = sName;
++            pItem->Handle = -1;
++            pItem->Value <<= sal_True;
++            pItem->State = beans::PropertyState_DIRECT_VALUE;
+         }
+     }
+ 
+-- 
+1.9.3
+
diff --git a/libreoffice.spec b/libreoffice.spec
index cbad381..2fc0ae6 100644
--- a/libreoffice.spec
+++ b/libreoffice.spec
@@ -324,6 +324,7 @@ Patch16: 0001-disable-PSD-import-test-which-deadlocks-on-ARM.patch
 Patch17: 0001-Resolves-fdo-37559-revert-adding-extra-dummy-polygon.patch
 Patch18: 0001-Resolves-fdo-86466-Wrong-background-color-shown-in-f.patch
 Patch19: 0001-Revert-Resolves-i63015-always-default-to-WinAnsiEnco.patch
+Patch20: 0001-Resolves-fdo-84043-don-t-create-duplicate-Mirrored-X.patch
 
 %define instdir %{_libdir}
 %define baseinstdir %{instdir}/libreoffice


More information about the scm-commits mailing list