[Libreoffice-commits] core.git: Branch 'libreoffice-7-4' - sfx2/source

[Christian Lohmaier (via logerrit)]

 sfx2/source/appl/sfxhelp.cxx |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

New commits:
commit 9988d022fffc24b1112ae066f6847b86e297667b
Author:     Christian Lohmaier <lohmaier+libreoff...@googlemail.com>
AuthorDate: Mon Nov 28 17:37:49 2022 +0100
Commit:     Caolán McNamara <caol...@redhat.com>
CommitDate: Tue Nov 29 11:45:16 2022 +0100

    f'up to the tdf#152172 fix re Safari as default browser on Ventura
    
    Previous patch did only adjust the check in the rarely used version of
    the function, not in the primarily used signature. D'oh!
    
    Change-Id: I019a74651d3ee8bab43a8c3997b8cd05040b7ba4
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/143410
    Tested-by: Jenkins
    Reviewed-by: Christian Lohmaier <lohmaier+libreoff...@googlemail.com>
    (cherry picked from commit c7b1fbff3673284d829606e5e07f9deb4258a5f5)
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/143273
    Reviewed-by: Caolán McNamara <caol...@redhat.com>

diff --git a/sfx2/source/appl/sfxhelp.cxx b/sfx2/source/appl/sfxhelp.cxx
index bcea21952dc5..8295e03926ba 100644
--- a/sfx2/source/appl/sfxhelp.cxx
+++ b/sfx2/source/appl/sfxhelp.cxx
@@ -1291,7 +1291,7 @@ bool SfxHelp::Start_Impl(const OUString& rURL, 
weld::Widget* pWidget, const OUSt
                                     
static_cast<CFStringRef>(@"https://www.libreoffice.org";),
                                     nullptr),
                                 kLSRolesAll, nullptr);
-        if([static_cast<NSString*>(CFURLGetString(pBrowser)) 
isEqualToString:@"file:///Applications/Safari.app/"]) {
+        if([static_cast<NSString*>(CFURLGetString(pBrowser)) 
hasSuffix:@"/Applications/Safari.app/"]) {
             impl_showOnlineHelp(aHelpURL, pWidget);
             return true;
         }