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

[Vasily Melenchuk (via logerrit)]

 sw/source/core/doc/DocumentLayoutManager.cxx |    9 +++++++++
 1 file changed, 9 insertions(+)

New commits:
commit a6c2a87612a10d55fc59fa01e44d67d95505834f
Author:     Vasily Melenchuk <vasily.melenchuk at cib.de>
AuthorDate: Thu Aug 20 12:06:03 2020 +0300
Commit:     Thorsten Behrens <Thorsten.Behrens at CIB.de>
CommitDate: Fri Aug 21 23:44:01 2020 +0200

    tdf#133490: set name for pasted frame format
    
    In some copy/paste operations resulting frame format name can
    be empty. This can lead to problems with udo/redo which relies
    on format names.
    
    Change-Id: I3cb542321d37cc00e2dee83d0f667164d83ead78
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/101042
    Tested-by: Jenkins
    Reviewed-by: Thorsten Behrens <Thorsten.Behrens at CIB.de>

diff --git a/sw/source/core/doc/DocumentLayoutManager.cxx b/sw/source/core/doc/DocumentLayoutManager.cxx
index a582fddaa1cb..7df5d2871f91 100644
--- a/sw/source/core/doc/DocumentLayoutManager.cxx
+++ b/sw/source/core/doc/DocumentLayoutManager.cxx
@@ -41,6 +41,7 @@
 #include <pam.hxx>
 #include <frameformats.hxx>
 #include <com/sun/star/embed/EmbedStates.hpp>
+#include <svx/svdobj.hxx>
 
 using namespace ::com::sun::star;
 
@@ -485,6 +486,14 @@ SwFrameFormat *DocumentLayoutManager::CopyLayoutFormat(
         pDestTextBox->SetOtherTextBoxFormat(pDest);
     }
 
+    if (pDest->GetName().isEmpty())
+    {
+        // Format name should have unique name. Let's use object name as a fallback
+        SdrObject *pObj = pDest->FindSdrObject();
+        if (pObj)
+            pDest->SetName(pObj->GetName());
+    }
+
     return pDest;
 }