[Libreoffice-commits] core.git: chart2/source

[Katarina Behrens]

 chart2/source/controller/itemsetwrapper/GraphicPropertyItemConverter.cxx |    
7 ++++++-
 1 file changed, 6 insertions(+), 1 deletion(-)

New commits:
commit 2eaa978d135a53abf22df2dc3fcb8d04733fe2d3
Author: Katarina Behrens <katarina.behr...@cib.de>
Date:   Wed Apr 4 13:29:09 2018 +0200

    tdf#104658: Fix jumping bitmap-filled chart bars
    
    Coincidentally this also resolves almost identical issues in chart wall
    (tdf#114132)
    
    Change-Id: Ib566110639d00aa46463ee592861ce3e15c64f50
    Reviewed-on: https://gerrit.libreoffice.org/52624
    Tested-by: Jenkins <c...@libreoffice.org>
    Reviewed-by: Katarina Behrens <katarina.behr...@cib.de>

diff --git 
a/chart2/source/controller/itemsetwrapper/GraphicPropertyItemConverter.cxx 
b/chart2/source/controller/itemsetwrapper/GraphicPropertyItemConverter.cxx
index 906318a6bdd6..198383cc2773 100644
--- a/chart2/source/controller/itemsetwrapper/GraphicPropertyItemConverter.cxx
+++ b/chart2/source/controller/itemsetwrapper/GraphicPropertyItemConverter.cxx
@@ -430,9 +430,14 @@ bool GraphicPropertyItemConverter::ApplySpecialItem(
                 bool bStretched = rItemSet.Get( XATTR_FILLBMP_STRETCH 
).GetValue();
                 drawing::BitmapMode aMode =
                     (bStretched ? drawing::BitmapMode_STRETCH : 
drawing::BitmapMode_NO_REPEAT);
+                drawing::BitmapMode aOtherMode = drawing::BitmapMode_NO_REPEAT;
 
                 aValue <<= aMode;
-                if( aValue != GetPropertySet()->getPropertyValue( 
aModePropName ))
+                GetPropertySet()->getPropertyValue( aModePropName ) >>= 
aOtherMode;
+
+                // don't overwrite if it has been set to BitmapMode_REPEAT (= 
tiled) already
+                // XATTR_FILLBMP_STRETCH and XATTR_FILLBMP_TILE often come in 
pairs, tdf#104658
+                if( aMode != aOtherMode && aOtherMode != 
drawing::BitmapMode_REPEAT )
                 {
                     GetPropertySet()->setPropertyValue( aModePropName, aValue 
);
                     bChanged = true;
_______________________________________________
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits