[Libreoffice-commits] core.git: include/svx svx/source

[Mark Hung (via logerrit)]

 include/svx/sdrundomanager.hxx       |    2 ++
 svx/source/svdraw/sdrundomanager.cxx |    3 +++
 svx/source/svdraw/svdedxv.cxx        |    3 ++-
 3 files changed, 7 insertions(+), 1 deletion(-)

New commits:
commit 7a641c71f8191e83bb6c408d3ff51a58d7dd4af9
Author:     Mark Hung <marklh9 at gmail.com>
AuthorDate: Wed Apr 14 22:30:06 2021 +0800
Commit:     Mark Hung <marklh9 at gmail.com>
CommitDate: Fri Apr 16 15:04:07 2021 +0200

    tdf#131634 Don't redo actions created before text edit begins.
    
    In SdrObjEditView::SdrEndTextEdit(), pSdrUndoManager->Redo() was
    invoked until all the redo actions created after text edit began
    were converted to undo actions.
    
    Without checking, all the redo actions include the ones created
    before text edit began were moved to undo stack, and caused the
    SdrTextObj to be destroyed in SdrUndoManager::SetEndTextEditHdl
    when removing the undo actions and a use after release problem.
    
    The patch add GetRedoActionCountBeforeTextEdit() so the program
    won't invoke pSdrUndoManager->Redo() on actions created before
    text edit begin.
    
    Change-Id: Ic010bc6e71ee78ef2cb20a5259dc9d6d6579ccaa
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/114102
    Tested-by: Jenkins
    Reviewed-by: Mark Hung <marklh9 at gmail.com>

diff --git a/include/svx/sdrundomanager.hxx b/include/svx/sdrundomanager.hxx
index 12fa1c7b3852..9ff23e441c54 100644
--- a/include/svx/sdrundomanager.hxx
+++ b/include/svx/sdrundomanager.hxx
@@ -33,6 +33,7 @@ private:
 
     Link<SdrUndoManager*, void> maEndTextEditHdl;
     SfxUndoAction* mpLastUndoActionBeforeTextEdit;
+    size_t mnRedoActionCountBeforeTextEdit;
     bool mbEndTextEditTriggeredFromUndo;
 
     SfxObjectShell* m_pDocSh;
@@ -64,6 +65,7 @@ public:
     // by a last undo during text edit
     bool isEndTextEditTriggeredFromUndo() const { return mbEndTextEditTriggeredFromUndo; }
     void SetDocShell(SfxObjectShell* pDocShell);
+    size_t GetRedoActionCountBeforeTextEdit() const { return mnRedoActionCountBeforeTextEdit; }
 };
 
 #endif // INCLUDED_SVX_SDRUNDOMANAGER_HXX
diff --git a/svx/source/svdraw/sdrundomanager.cxx b/svx/source/svdraw/sdrundomanager.cxx
index 1e64a1a89482..8b1f19fbecfa 100644
--- a/svx/source/svdraw/sdrundomanager.cxx
+++ b/svx/source/svdraw/sdrundomanager.cxx
@@ -25,6 +25,7 @@ SdrUndoManager::SdrUndoManager()
     : EditUndoManager(20 /*nMaxUndoActionCount*/)
     , maEndTextEditHdl()
     , mpLastUndoActionBeforeTextEdit(nullptr)
+    , mnRedoActionCountBeforeTextEdit(0)
     , mbEndTextEditTriggeredFromUndo(false)
     , m_pDocSh(nullptr)
 {
@@ -107,6 +108,7 @@ void SdrUndoManager::SetEndTextEditHdl(const Link<SdrUndoManager*, void>& rLink)
     {
         // text edit start, remember last non-textedit action for later cleanup
         mpLastUndoActionBeforeTextEdit = GetUndoActionCount() ? GetUndoAction() : nullptr;
+        mnRedoActionCountBeforeTextEdit = GetRedoActionCount();
     }
     else
     {
@@ -123,6 +125,7 @@ void SdrUndoManager::SetEndTextEditHdl(const Link<SdrUndoManager*, void>& rLink)
 
         // forget marker again
         mpLastUndoActionBeforeTextEdit = nullptr;
+        mnRedoActionCountBeforeTextEdit = 0;
     }
 }
 
diff --git a/svx/source/svdraw/svdedxv.cxx b/svx/source/svdraw/svdedxv.cxx
index e18738dba12a..a4ae54035c2c 100644
--- a/svx/source/svdraw/svdedxv.cxx
+++ b/svx/source/svdraw/svdedxv.cxx
@@ -1441,7 +1441,8 @@ SdrEndTextEditKind SdrObjEditView::SdrEndTextEdit(bool bDontDeleteReally)
                     // to create a complete text change undo action for the redo buffer. Also mark this
                     // state when at least one redo was executed; the created extra TextChange needs to
                     // be undone in addition to the first real undo outside the text edit changes
-                    while (pSdrUndoManager->GetRedoActionCount())
+                    while (pSdrUndoManager->GetRedoActionCount()
+                           > pSdrUndoManager->GetRedoActionCountBeforeTextEdit())
                     {
                         bNeedToUndoSavedRedoTextEdit = true;
                         pSdrUndoManager->Redo();