Don't select hyperlink fields arbitrarily

For some functions, like remove or edit hyperlink, it's required
that the hyperlink field is selected.
However, we should do that only when executing the function, not
when queriying its status.
This led to selection changes on mouse over/right click.

Change-Id: I22035698032d1e651aae6a74d6e445dfe044bf74
Reviewed-on: https://gerrit.libreoffice.org/77336
Tested-by: Jenkins
Reviewed-by: Samuel Mehrbrodt <Samuel.Mehrbrodt@cib.de>

1 files changed, 5 insertions, 1 deletions

diff --git a/sd/source/ui/view/drviews2.cxx b/sd/source/ui/view/drviews2.cxx
index ed6ca12113e3..e81ff4b62bce 100644
--- a/sd/source/ui/view/drviews2.cxx
+++ b/sd/source/ui/view/drviews2.cxx
@@ -1154,6 +1154,10 @@ void DrawViewShell::FuTemporary(SfxRequest& rReq)
         {
             if (mpDrawView->IsTextEdit())
             {
+                // First make sure the field is selected
+                OutlinerView* pOutView = mpDrawView->GetTextEditOutlinerView();
+                if (pOutView)
+                    pOutView->SelectFieldAtCursor();
                 URLFieldHelper::RemoveURLField(mpDrawView->GetTextEditOutliner(),
                                                mpDrawView->GetTextEditOutlinerView());
             }
@@ -2119,7 +2123,7 @@ void DrawViewShell::FuTemporary(SfxRequest& rReq)
             // Ensure the field is selected first
             OutlinerView* pOutView = mpDrawView->GetTextEditOutlinerView();
             if (pOutView)
-                pOutView->GetFieldAtCursor();
+                pOutView->SelectFieldAtCursor();
 
             GetViewFrame()->GetDispatcher()->Execute( SID_HYPERLINK_DIALOG );