sw/source/core/access/AccessibilityCheck.cxx |   14 ++++++++++----
 1 file changed, 10 insertions(+), 4 deletions(-)

New commits:
commit 585b2cec2f6441a698588e5a27d02c14378daf49
Author:     Balazs Varga <balazs.varga.ext...@allotropia.de>
AuthorDate: Wed Aug 2 13:05:37 2023 +0200
Commit:     Michael Stahl <michael.st...@allotropia.de>
CommitDate: Mon Aug 7 12:21:12 2023 +0200

    tdf#156568 - A11Y - Make clickable the "Hyperlink text is the same as..."
    
    warning message.
    
    Change-Id: Iad391e14f3f414bd4674e634a160bd749b281197
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/155225
    Tested-by: Jenkins
    Reviewed-by: Balazs Varga <balazs.varga.ext...@allotropia.de>
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/155356
    Reviewed-by: Michael Stahl <michael.st...@allotropia.de>

diff --git a/sw/source/core/access/AccessibilityCheck.cxx 
b/sw/source/core/access/AccessibilityCheck.cxx
index 2d44e8689b58..80fc5624d7ed 100644
--- a/sw/source/core/access/AccessibilityCheck.cxx
+++ b/sw/source/core/access/AccessibilityCheck.cxx
@@ -332,18 +332,24 @@ private:
         {
             OUString sText = xTextRange->getString();
             INetURLObject aHyperlink(sHyperlink);
+            std::shared_ptr<sw::AccessibilityIssue> pIssue;
+
             if (aHyperlink.GetProtocol() != INetProtocol::NotValid
                 && INetURLObject(sText) == aHyperlink)
             {
                 OUString sIssueText
                     = 
SwResId(STR_HYPERLINK_TEXT_IS_LINK).replaceFirst("%LINK%", sHyperlink);
-                lclAddIssue(m_rIssueCollection, sIssueText,
-                            sfx::AccessibilityIssueID::HYPERLINK_IS_TEXT);
+                pIssue = lclAddIssue(m_rIssueCollection, sIssueText,
+                                     
sfx::AccessibilityIssueID::HYPERLINK_IS_TEXT);
             }
             else if (sText.getLength() <= 5)
             {
-                auto pIssue = lclAddIssue(m_rIssueCollection, 
SwResId(STR_HYPERLINK_TEXT_IS_SHORT),
-                                          
sfx::AccessibilityIssueID::HYPERLINK_SHORT);
+                pIssue = lclAddIssue(m_rIssueCollection, 
SwResId(STR_HYPERLINK_TEXT_IS_SHORT),
+                                     
sfx::AccessibilityIssueID::HYPERLINK_SHORT);
+            }
+
+            if (pIssue)
+            {
                 pIssue->setIssueObject(IssueObject::TEXT);
                 pIssue->setNode(pTextNode);
                 SwDoc& rDocument = pTextNode->GetDoc();

Reply via email to