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

Caolán McNamara caolanm at redhat.com
Thu May 31 08:14:48 UTC 2018


 chart2/source/controller/main/ChartController_Tools.cxx |    3 ++-
 chart2/source/controller/sidebar/ChartSeriesPanel.cxx   |    4 ++++
 2 files changed, 6 insertions(+), 1 deletion(-)

New commits:
commit 0b40212c18bb3c518bf54e55b9060f908ed84ed1
Author: Caolán McNamara <caolanm at redhat.com>
Date:   Wed May 30 13:57:02 2018 +0100

    tdf#117864 TransferableHelper::GetSystemClipboard requires a focus window
    
    Change-Id: Ibd9b9218cf12e80db1fc9a70bdbab38d3df950f7
    Reviewed-on: https://gerrit.libreoffice.org/55073
    Reviewed-by: Caolán McNamara <caolanm at redhat.com>
    Tested-by: Caolán McNamara <caolanm at redhat.com>

diff --git a/chart2/source/controller/main/ChartController_Tools.cxx b/chart2/source/controller/main/ChartController_Tools.cxx
index a89f2ef70821..59444c95d5fb 100644
--- a/chart2/source/controller/main/ChartController_Tools.cxx
+++ b/chart2/source/controller/main/ChartController_Tools.cxx
@@ -524,7 +524,8 @@ void ChartController::executeDispatch_Copy()
             }
             if ( xTransferable.is() )
             {
-                Reference< datatransfer::clipboard::XClipboard > xClipboard( TransferableHelper::GetSystemClipboard() );
+                SolarMutexGuard aSolarGuard;
+                Reference<datatransfer::clipboard::XClipboard> xClipboard(GetChartWindow()->GetClipboard());
                 if ( xClipboard.is() )
                 {
                     xClipboard->setContents( xTransferable, Reference< datatransfer::clipboard::XClipboardOwner >() );
diff --git a/chart2/source/controller/sidebar/ChartSeriesPanel.cxx b/chart2/source/controller/sidebar/ChartSeriesPanel.cxx
index 836aaaba3002..edf0190d75fc 100644
--- a/chart2/source/controller/sidebar/ChartSeriesPanel.cxx
+++ b/chart2/source/controller/sidebar/ChartSeriesPanel.cxx
@@ -266,6 +266,10 @@ OUString getCID(const css::uno::Reference<css::frame::XModel>& xModel)
 
     OUString aCID;
     aAny >>= aCID;
+
+    if (aCID.isEmpty())
+        return OUString();
+
 #ifdef DBG_UTIL
     ObjectType eType = ObjectIdentifier::getObjectType(aCID);
     assert(eType == OBJECTTYPE_DATA_SERIES);


More information about the Libreoffice-commits mailing list