[Libreoffice-commits] core.git: Branch 'distro/collabora/cp-6.2' - chart2/source

[Szymon KÅ‚os (via logerrit)]

 chart2/source/controller/main/ChartController.cxx |    5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

New commits:
commit 2be854e8441b870a2e9bed2d583d4f7aeb73d5d9
Author:     Szymon Kłos <szymon.klos at collabora.com>
AuthorDate: Tue Oct 6 16:59:52 2020 +0200
Commit:     Andras Timar <andras.timar at collabora.com>
CommitDate: Wed Oct 28 13:36:23 2020 +0100

    Fire selection change event before sidebar is unregistered
    
    Prevent us from crash in online using mobile phone
    when leaving chart edit mode with chart type modified.
    
    Change-Id: I8da70612786141a2c92b55a45822ef8ff5713e5b
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/104035
    Tested-by: Jenkins CollaboraOffice <jenkinscollaboraoffice at gmail.com>
    Reviewed-by: Andras Timar <andras.timar at collabora.com>

diff --git a/chart2/source/controller/main/ChartController.cxx b/chart2/source/controller/main/ChartController.cxx
index 3d88f3aadd1a..fb1ed3efabc1 100644
--- a/chart2/source/controller/main/ChartController.cxx
+++ b/chart2/source/controller/main/ChartController.cxx
@@ -754,6 +754,9 @@ void SAL_CALL ChartController::dispose()
 {
     m_bDisposed = true;
 
+    mpSelectionChangeHandler->selectionChanged(css::lang::EventObject());
+    mpSelectionChangeHandler->Disconnect();
+
     if (getModel().is())
     {
         uno::Reference<ui::XSidebar> xSidebar = getSidebarFromModel(getModel());
@@ -762,8 +765,6 @@ void SAL_CALL ChartController::dispose()
             sfx2::sidebar::SidebarController::unregisterSidebarForFrame(pSidebar, this);
         }
     }
-    mpSelectionChangeHandler->selectionChanged(css::lang::EventObject());
-    mpSelectionChangeHandler->Disconnect();
 
     try
     {