commit 90b12c9bad55e8f50b75a6d7b68caa27d82cc2b9
author Justin Luth <jluth@mail.com> Fri Dec 22 14:59:47 2023 -0500
committer Justin Luth <jluth@mail.com> Sat Dec 23 00:51:52 2023 +0100
tree 4f9398f2636f3a6d2abb3c36dc630672a9f9c705
parent 73102542754189e3140f40f8cf02b60d6268dc0a

tdf#111969 sw: add selected smarttag to context menu

If the smart tag was selected (from start to end),
then GetPoint() was returning the end position
which is considered to be outside of the smarttag.

Instead, use Start(), which doesn't care which
direction the selection happened in.

Smart tags REQUIRE an extension, so no unit test possible.

Change-Id: I3ad2f686a9f5d6accb0d1ac2ffbf272260249223
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/161194
Tested-by: Jenkins
Reviewed-by: Justin Luth <jluth@mail.com>

sw/source/core/crsr/crsrsh.cxx

diff --git a/sw/source/core/crsr/crsrsh.cxx b/sw/source/core/crsr/crsrsh.cxx
index e910a3a..449b074 100644
--- a/sw/source/core/crsr/crsrsh.cxx
+++ b/sw/source/core/crsr/crsrsh.cxx
@@ -4109,7 +4109,7 @@ void SwCursorShell::GetSmartTagTerm( std::vector< OUString >& rSmartTagTypes,
        return;

    SwPaM* pCursor = GetCursor();
    SwPosition aPos( *pCursor->GetPoint() );
    SwPosition aPos(*pCursor->Start());
    SwTextNode *pNode = aPos.GetNode().GetTextNode();
    if ( !pNode || pNode->IsInProtectSect() )
        return;