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

Dennis Francis dennis.francis at collabora.co.uk
Wed Aug 2 06:53:03 UTC 2017


 chart2/source/model/main/ChartModel.cxx |    7 +++++++
 1 file changed, 7 insertions(+)

New commits:
commit 9561e0f392de3b99d8a0374486d4c0575b23180b
Author: Dennis Francis <dennis.francis at collabora.co.uk>
Date:   Wed Aug 2 10:38:12 2017 +0530

    tdf#109523: Do not forget to remove the chart model from...
    
    data-provider's listener set when the chart model is
    getting disposed.
    
    Change-Id: Iafbe1426cdb42059cbb4e2955a257a09aa42ec8e
    Reviewed-on: https://gerrit.libreoffice.org/40663
    Reviewed-by: Markus Mohrhard <markus.mohrhard at googlemail.com>
    Tested-by: Jenkins <ci at libreoffice.org>

diff --git a/chart2/source/model/main/ChartModel.cxx b/chart2/source/model/main/ChartModel.cxx
index 8918c04cc2a4..a73aa04f49ef 100644
--- a/chart2/source/model/main/ChartModel.cxx
+++ b/chart2/source/model/main/ChartModel.cxx
@@ -548,6 +548,13 @@ void SAL_CALL ChartModel::dispose()
     if ( m_xDiagram.is() )
         ModifyListenerHelper::removeListener( m_xDiagram, this );
 
+    if ( m_xDataProvider.is() )
+    {
+        Reference<util::XModifyBroadcaster> xModifyBroadcaster( m_xDataProvider, uno::UNO_QUERY );
+        if ( xModifyBroadcaster.is() )
+            xModifyBroadcaster->removeModifyListener( this );
+    }
+
     m_xDataProvider.clear();
     m_xInternalDataProvider.clear();
     m_xNumberFormatsSupplier.clear();


More information about the Libreoffice-commits mailing list