[Libreoffice-commits] core.git: Branch 'libreoffice-6-2' - 2 commits - sfx2/source
Libreoffice Gerrit user
logerrit at kemper.freedesktop.org
Sun Dec 2 13:34:26 UTC 2018
sfx2/source/appl/workwin.cxx | 3 ++-
sfx2/source/notebookbar/SfxNotebookBar.cxx | 11 ++++++++++-
2 files changed, 12 insertions(+), 2 deletions(-)
New commits:
commit 53170cae2057f705e779ec4b8538d422d071986e
Author: Mike Kaganski <mike.kaganski at collabora.com>
AuthorDate: Thu Nov 29 21:18:57 2018 +0100
Commit: Xisco Faulí <xiscofauli at libreoffice.org>
CommitDate: Sun Dec 2 14:34:16 2018 +0100
Revert "Revert "Notebookbar:Disabled appearence of context toolbars""
This reverts commit 9a6169529eb687190531225bfe9fef850325ee9e now
that proper fix for both tdf#121591 and tdf#119945 is pushed -
see commit 0a36be3fee1b4f1b47b55a64668b6aa99896ab24.
Change-Id: Ia44510d427e0626a5f566210247db8848653f8e2
Reviewed-on: https://gerrit.libreoffice.org/64279
Tested-by: Jenkins
Reviewed-by: Mike Kaganski <mike.kaganski at collabora.com>
(cherry picked from commit f3ea5e21bea80004b04edbf0287d4a2e1aaef5d0)
Reviewed-on: https://gerrit.libreoffice.org/64316
Reviewed-by: Xisco Faulí <xiscofauli at libreoffice.org>
diff --git a/sfx2/source/appl/workwin.cxx b/sfx2/source/appl/workwin.cxx
index 5320d832b13a..b7f6eddd9777 100644
--- a/sfx2/source/appl/workwin.cxx
+++ b/sfx2/source/appl/workwin.cxx
@@ -55,6 +55,7 @@
#include <com/sun/star/lang/DisposedException.hpp>
#include <type_traits>
#include <unordered_map>
+#include <sfx2/notebookbar/SfxNotebookBar.hxx>
using namespace ::com::sun::star;
using namespace ::com::sun::star::uno;
@@ -1203,7 +1204,7 @@ void SfxWorkWindow::UpdateObjectBars_Impl2()
// Is a ToolBox required in this context ?
bool bModesMatching = (nUpdateMode != SfxVisibilityFlags::Invisible) && ((nTbxMode & nUpdateMode) == nUpdateMode);
- if ( bDestroy )
+ if ( bDestroy || sfx2::SfxNotebookBar::IsActive())
{
OUString aTbxId = g_aTbxTypeName + GetResourceURLFromToolbarId(eId);
xLayoutManager->destroyElement( aTbxId );
commit 9ce21daf20faae0e5c8dd8eeb0909106e23a4319
Author: Mike Kaganski <mike.kaganski at collabora.com>
AuthorDate: Thu Nov 29 17:17:15 2018 +0300
Commit: Xisco Faulí <xiscofauli at libreoffice.org>
CommitDate: Sun Dec 2 14:34:04 2018 +0100
tdf#121591, tdf#119945: ModuleManager::identify throws for embedded modules
Change-Id: Ief1a664fdf26c837fd55c9a236f45aa045067309
Reviewed-on: https://gerrit.libreoffice.org/64256
Tested-by: Jenkins
Reviewed-by: Mike Kaganski <mike.kaganski at collabora.com>
(cherry picked from commit 0a36be3fee1b4f1b47b55a64668b6aa99896ab24)
Reviewed-on: https://gerrit.libreoffice.org/64280
Reviewed-by: Xisco Faulí <xiscofauli at libreoffice.org>
diff --git a/sfx2/source/notebookbar/SfxNotebookBar.cxx b/sfx2/source/notebookbar/SfxNotebookBar.cxx
index 953bacf4bfe8..5dcaea7861d8 100644
--- a/sfx2/source/notebookbar/SfxNotebookBar.cxx
+++ b/sfx2/source/notebookbar/SfxNotebookBar.cxx
@@ -17,6 +17,7 @@
#include <vcl/tabctrl.hxx>
#include <sfx2/viewfrm.hxx>
#include <comphelper/processfactory.hxx>
+#include <com/sun/star/frame/UnknownModuleException.hpp>
#include <com/sun/star/ui/ContextChangeEventMultiplexer.hpp>
#include <com/sun/star/ui/XContextChangeEventMultiplexer.hpp>
#include <com/sun/star/util/URLTransformer.hpp>
@@ -208,7 +209,15 @@ bool SfxNotebookBar::IsActive()
return false;
const Reference<frame::XModuleManager> xModuleManager = frame::ModuleManager::create( ::comphelper::getProcessComponentContext() );
- eApp = vcl::EnumContext::GetApplicationEnum(xModuleManager->identify(xFrame));
+ try
+ {
+ eApp = vcl::EnumContext::GetApplicationEnum(xModuleManager->identify(xFrame));
+ }
+ catch (css::frame::UnknownModuleException& e)
+ {
+ SAL_WARN("sfx.appl", "SfxNotebookBar::IsActive(): " + e.Message);
+ return false;
+ }
}
else
return false;
More information about the Libreoffice-commits
mailing list