[Libreoffice-commits] core.git: Branch 'libreoffice-6-0-7' - sd/source

[Libreoffice Gerrit user]

 sd/source/core/stlpool.cxx |    5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

New commits:
commit ea3e84429f84c4a57ce28980b799cc6d6995ef26
Author:     Michael Stahl <Michael.Stahl at cib.de>
AuthorDate: Sat Oct 27 19:56:00 2018 +0200
Commit:     Caolán McNamara <caolanm at redhat.com>
CommitDate: Tue Oct 30 13:43:52 2018 +0100

    tdf#120376 sd: fix duplicated styles on copy/paste
    
    Unfortunately the comparison was inverted, so a style is copied
    iff it already exists, which is clearly the reviewer's fault...
    
    (regression from 57db6e24b5ad43d447c30e44a112c74c7e75b46b)
    
    Change-Id: I3425982feb08e980eca9243cc16120897b65a70f
    Reviewed-on: https://gerrit.libreoffice.org/62436
    Tested-by: Jenkins
    Reviewed-by: Michael Stahl <Michael.Stahl at cib.de>
    (cherry picked from commit be9f3db2306150a37ef18e4ccc8d8f4a1934c5c1)
    Reviewed-on: https://gerrit.libreoffice.org/62452
    Reviewed-by: Armin Le Grand <Armin.Le.Grand at cib.de>
    (cherry picked from commit ba5d56afd88ba29ba7f222d15f4b9046011cc38e)
    Reviewed-on: https://gerrit.libreoffice.org/62471
    Reviewed-by: Adolfo Jayme Barrientos <fitojb at ubuntu.com>
    Tested-by: Xisco Faulí <xiscofauli at libreoffice.org>
    Reviewed-by: Thorsten Behrens <Thorsten.Behrens at CIB.de>
    Reviewed-by: Caolán McNamara <caolanm at redhat.com>
    Tested-by: Caolán McNamara <caolanm at redhat.com>

diff --git a/sd/source/core/stlpool.cxx b/sd/source/core/stlpool.cxx
index c627970fd4e2..aa3d1de8a89a 100644
--- a/sd/source/core/stlpool.cxx
+++ b/sd/source/core/stlpool.cxx
@@ -652,7 +652,7 @@ void SdStyleSheetPool::CopySheets(SdStyleSheetPool& rSourcePool, SfxStyleFamily
             pExistingSheet =
                 GetStyleSheetByPositionInIndex(aSheetsWithName.front()).get();
             if (!rRenameSuffix.isEmpty() &&
-                pExistingSheet->GetItemSet().Equals(xSheet->GetItemSet(), false))
+                !pExistingSheet->GetItemSet().Equals(xSheet->GetItemSet(), false))
             {
                 // we have found a sheet with the same name, but different contents. Try to find a new name.
                 // If we already have a sheet with the new name, and it is equal to the one in the source pool,
@@ -664,7 +664,8 @@ void SdStyleSheetPool::CopySheets(SdStyleSheetPool& rSourcePool, SfxStyleFamily
                     aTmpName = aName + rRenameSuffix + OUString::number(nSuffix);
                     pExistingSheet = Find(aTmpName, eFamily);
                     nSuffix++;
-                } while( pExistingSheet && pExistingSheet->GetItemSet().Equals(xSheet->GetItemSet(), false) );
+                } while (pExistingSheet &&
+                        !pExistingSheet->GetItemSet().Equals(xSheet->GetItemSet(), false));
                 aName = aTmpName;
                 bAddToList = true;
             }