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

[Caolán McNamara (via logerrit)]

 sfx2/source/appl/openuriexternally.cxx |    5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

New commits:
commit 518df6a3bd6aa251dc4ad30ab3d04545513ae5c6
Author:     Caolán McNamara <caolan.mcnam...@collabora.com>
AuthorDate: Tue Nov 7 12:35:47 2023 +0000
Commit:     Caolán McNamara <caolan.mcnam...@collabora.com>
CommitDate: Tue Nov 7 14:53:08 2023 +0100

    make this a more straight forward yes/no question
    
    instead of an ok/cancel, and default to no
    
    Change-Id: If86ca1b29d0dd91018fdc8edd6482a6d9ef5c0e2
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/159068
    Tested-by: Jenkins
    Reviewed-by: Caolán McNamara <caolan.mcnam...@collabora.com>

diff --git a/sfx2/source/appl/openuriexternally.cxx 
b/sfx2/source/appl/openuriexternally.cxx
index 680c2796bafa..bdf8d72766cc 100644
--- a/sfx2/source/appl/openuriexternally.cxx
+++ b/sfx2/source/appl/openuriexternally.cxx
@@ -94,10 +94,11 @@ IMPL_LINK_NOARG(URITools, onOpenURI, Timer*, void)
             SolarMutexGuard g;
             std::unique_ptr<weld::MessageDialog> eb(
                 Application::CreateMessageDialog(
-                    mpDialogParent, VclMessageType::Warning, 
VclButtonsType::OkCancel,
+                    mpDialogParent, VclMessageType::Warning, 
VclButtonsType::YesNo,
                     SfxResId(STR_DANGEROUS_TO_OPEN)));
             
eb->set_primary_text(eb->get_primary_text().replaceFirst("$(ARG1)", 
INetURLObject::decode(msURI, INetURLObject::DecodeMechanism::Unambiguous)));
-            if (eb->run() == RET_OK) {
+            eb->set_default_response(RET_NO);
+            if (eb->run() == RET_YES) {
                 flags = 0;
                 continue;
             }