[Libreoffice-commits] core.git: Branch 'libreoffice-4-4' - svx/source

[Caolán McNamara]

 svx/source/svdraw/svdedtv1.cxx |   38 ++++++++++++++++++++++----------------
 svx/source/table/cell.cxx      |   15 +++++++++++++--
 2 files changed, 35 insertions(+), 18 deletions(-)

New commits:
commit d97c984dc016e14a9f16a3ae0b865e3f2b78fbc8
Author: Caolán McNamara <caolanm at redhat.com>
Date:   Wed Feb 18 15:04:45 2015 +0000

    Resolves: rhbz#1193971 clear hard-coded char props in table cells...
    
    like we do already for shapes. So now when attempting to set replacement char
    properties on the entire cell, clear any hard-coded sub ranges within the
    existing paragraphs.
    
    This extends the effort of
    commit a217f3cb2dce71a4322f78ceb45edb6f171b2b65
    Author: Matúš Kukan <matus.kukan at collabora.com>
    Date:   Tue Aug 26 10:16:34 2014 +0200
    
        bnc#770711: Make changing table text color in impress always work
    
    to encompass all character properties, not just color
    
    Change-Id: If8b426c3531b4678868d8351426db32c3c558650
    (cherry picked from commit b14259b946686c753c0df42c519fc2537e871601)
    Reviewed-on: https://gerrit.libreoffice.org/14539
    Reviewed-by: Miklos Vajna <vmiklos at collabora.co.uk>
    Tested-by: Miklos Vajna <vmiklos at collabora.co.uk>

diff --git a/svx/source/svdraw/svdedtv1.cxx b/svx/source/svdraw/svdedtv1.cxx
index 1ba5f40..62c4cff 100644
--- a/svx/source/svdraw/svdedtv1.cxx
+++ b/svx/source/svdraw/svdedtv1.cxx
@@ -953,6 +953,26 @@ void SdrEditView::MergeAttrFromMarked(SfxItemSet& rAttr, bool bOnlyHardAttr) con
     }
 }
 
+std::vector<sal_uInt16> GetAllCharPropIds(const SfxItemSet& rSet)
+{
+    std::vector<sal_uInt16> aCharWhichIds;
+    {
+        SfxItemIter aIter(rSet);
+        const SfxPoolItem* pItem=aIter.FirstItem();
+        while (pItem!=NULL)
+        {
+            if (!IsInvalidItem(pItem))
+            {
+                sal_uInt16 nWhich = pItem->Which();
+                if (nWhich>=EE_CHAR_START && nWhich<=EE_CHAR_END)
+                    aCharWhichIds.push_back( nWhich );
+            }
+            pItem=aIter.NextItem();
+        }
+    }
+    return aCharWhichIds;
+}
+
 void SdrEditView::SetAttrToMarked(const SfxItemSet& rAttr, bool bReplaceAll)
 {
     if (AreObjectsMarked())
@@ -977,25 +997,11 @@ void SdrEditView::SetAttrToMarked(const SfxItemSet& rAttr, bool bReplaceAll)
         }
 #endif
 
-        // #103836# if the user thets character attributes to the complete shape,
+        // #103836# if the user sets character attributes to the complete shape,
         //          we want to remove all hard set character attributes with same
         //          which ids from the text. We do that later but here we remember
         //          all character attribute which id's that are set.
-        std::vector<sal_uInt16> aCharWhichIds;
-        {
-            SfxItemIter aIter(rAttr);
-            const SfxPoolItem* pItem=aIter.FirstItem();
-            while( pItem!=NULL )
-            {
-                if (!IsInvalidItem(pItem))
-                {
-                    sal_uInt16 nWhich = pItem->Which();
-                    if (nWhich>=EE_CHAR_START && nWhich<=EE_CHAR_END)
-                        aCharWhichIds.push_back( nWhich );
-                }
-                pItem=aIter.NextItem();
-            }
-        }
+        std::vector<sal_uInt16> aCharWhichIds(GetAllCharPropIds(rAttr));
 
         // To make Undo reconstruct text attributes correctly after Format.Standard
         bool bHasEEItems=SearchOutlinerItems(rAttr,bReplaceAll);
diff --git a/svx/source/table/cell.cxx b/svx/source/table/cell.cxx
index e81370c..3468283 100644
--- a/svx/source/table/cell.cxx
+++ b/svx/source/table/cell.cxx
@@ -139,6 +139,8 @@ SdrText* CellTextProvider::getText(sal_Int32 nIndex) const
 
 }
 
+extern std::vector<sal_uInt16> GetAllCharPropIds(const SfxItemSet& rSet);
+
 namespace sdr
 {
     namespace properties
@@ -261,12 +263,21 @@ namespace sdr
 
                     sal_Int32 nParaCount(pOutliner->GetParagraphCount());
 
+                    // if the user sets character attributes to the complete
+                    // cell we want to remove all hard set character attributes
+                    // with same which ids from the text
+                    std::vector<sal_uInt16> aCharWhichIds(GetAllCharPropIds(rSet));
+
                     for(sal_Int32 nPara = 0; nPara < nParaCount; nPara++)
                     {
                         SfxItemSet aSet(pOutliner->GetParaAttribs(nPara));
                         aSet.Put(rSet);
-                        if (aSet.GetItemState(EE_CHAR_COLOR, false) == SfxItemState::SET)
-                            pOutliner->RemoveCharAttribs( nPara, EE_CHAR_COLOR );
+
+                        for (std::vector<sal_uInt16>::const_iterator aI = aCharWhichIds.begin(); aI != aCharWhichIds.end(); ++aI)
+                        {
+                            pOutliner->RemoveCharAttribs(nPara, *aI);
+                        }
+
                         pOutliner->SetParaAttribs(nPara, aSet);
                     }