core.git: sfx2/source

[Gabor Kelemen (via logerrit)]

 sfx2/source/dialog/backingwindow.cxx |   30 +++++++++---------------------
 1 file changed, 9 insertions(+), 21 deletions(-)

New commits:
commit ad7e95531d791e9c16a38aeeeb17ecaea44d69e3
Author:     Gabor Kelemen <gabor.kelemen.ext...@allotropia.de>
AuthorDate: Mon May 13 09:14:06 2024 +0200
Commit:     Gabor Kelemen <gabor.kelemen.ext...@allotropia.de>
CommitDate: Wed May 15 08:58:04 2024 +0200

    Drop unnecessary ConfigurationAccess
    
    Change-Id: I8ae3273ab048df66fa6a7006cb00af7a4958e337
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/167569
    Reviewed-by: Gabor Kelemen <gabor.kelemen.ext...@allotropia.de>
    Tested-by: Jenkins

diff --git a/sfx2/source/dialog/backingwindow.cxx 
b/sfx2/source/dialog/backingwindow.cxx
index 0cb8bc258023..e3761327ee60 100644
--- a/sfx2/source/dialog/backingwindow.cxx
+++ b/sfx2/source/dialog/backingwindow.cxx
@@ -62,8 +62,6 @@ using namespace ::com::sun::star::frame;
 using namespace ::com::sun::star::uno;
 using namespace ::com::sun::star::document;
 
-constexpr OUStringLiteral SERVICENAME_CFGREADACCESS = 
u"com.sun.star.configuration.ConfigurationAccess";
-
 class BrandImage final : public weld::CustomWidgetController
 {
 private:
@@ -535,26 +533,16 @@ IMPL_LINK(BackingWindow, ExtLinkClickHdl, weld::Button&, 
rButton,void)
 
     try
     {
-        uno::Sequence<uno::Any> args(comphelper::InitAnyPropertySequence(
-        {
-            {"nodepath", 
uno::Any(u"/org.openoffice.Office.Common/Help/StartCenter"_ustr)}
-        }));
+        OUString sURL(officecfg::Office::Common::Menus::ExtensionsURL::get() +
+            "?LOvers=" + utl::ConfigManager::getProductVersion() +
+            "&LOlocale=" + 
LanguageTag(utl::ConfigManager::getUILocale()).getBcp47() );
 
-        Reference<lang::XMultiServiceFactory> xConfig = 
configuration::theDefaultProvider::get( 
comphelper::getProcessComponentContext() );
-        Reference<container::XNameAccess> 
xNameAccess(xConfig->createInstanceWithArguments(SERVICENAME_CFGREADACCESS, 
args), UNO_QUERY);
-        if (xNameAccess.is())
-        {
-            OUString 
sURL(officecfg::Office::Common::Menus::ExtensionsURL::get() +
-                "?LOvers=" + utl::ConfigManager::getProductVersion() +
-                "&LOlocale=" + 
LanguageTag(utl::ConfigManager::getUILocale()).getBcp47() );
-
-            Reference<css::system::XSystemShellExecute> const
-                xSystemShellExecute(
-                    css::system::SystemShellExecute::create(
-                        ::comphelper::getProcessComponentContext()));
-            xSystemShellExecute->execute(sURL, OUString(),
-                css::system::SystemShellExecuteFlags::URIS_ONLY);
-        }
+        Reference<css::system::XSystemShellExecute> const
+            xSystemShellExecute(
+                css::system::SystemShellExecute::create(
+                    ::comphelper::getProcessComponentContext()));
+        xSystemShellExecute->execute(sURL, OUString(),
+            css::system::SystemShellExecuteFlags::URIS_ONLY);
     }
     catch (const Exception&)
     {