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

Tamás Zolnai tamas.zolnai at collabora.com
Thu Dec 28 13:01:33 UTC 2017


 sfx2/source/sidebar/UnoPanels.cxx |    3 +++
 1 file changed, 3 insertions(+)

New commits:
commit 23a83639f5a06708074b13bc13ea4f6f819f55f0
Author: Tamás Zolnai <tamas.zolnai at collabora.com>
Date:   Thu Dec 28 02:38:52 2017 +0100

    tdf#104870 - Impress crashes switching views in read-only mode
    
    We need to check read only mode similar to SidebarController::CreatePanels()
    method. Otherwise SfxUnoPanels::getByName() creates an invalid panel.
    
    Change-Id: Ib7801b81c95f3f505a06c00f749ba4ed5809bfe0
    Reviewed-on: https://gerrit.libreoffice.org/47116
    Tested-by: Xisco Faulí <xiscofauli at libreoffice.org>
    Reviewed-by: Tamás Zolnai <tamas.zolnai at collabora.com>

diff --git a/sfx2/source/sidebar/UnoPanels.cxx b/sfx2/source/sidebar/UnoPanels.cxx
index a74763572cdb..49815a9ff649 100644
--- a/sfx2/source/sidebar/UnoPanels.cxx
+++ b/sfx2/source/sidebar/UnoPanels.cxx
@@ -108,6 +108,9 @@ sal_Bool SAL_CALL SfxUnoPanels::hasByName( const OUString& aName )
             iPanel(aPanels.begin()), iEnd(aPanels.end());
             iPanel!=iEnd; ++iPanel)
             {
+                // Determine if the panel can be displayed.
+                if(pSidebarController->IsDocumentReadOnly() && !iPanel->mbShowForReadOnlyDocuments)
+                    continue;
                 if (iPanel->msId == aName)
                     return true;
             }


More information about the Libreoffice-commits mailing list