? ? Pending

User tests: Successful: Unsuccessful:

avatar chmst
chmst
14 Sep 2021

Pull Request for Issue #35514 (comment)
related to #35548

Summary of Changes

We use to code Links where they are needed and for readability we use to break lines.
This causes errors in the output if a link is followed by a text, as the EOL is part of the link.

In mod_syndication I used HtmlHelper for generating the link.

Testing Instructions

See the generated link in the module syndication befor the patch (easy if you have blog sample data).
Apply the patch and the link is underlined properly.

Actual result BEFORE applying this Pull Request

Expected result AFTER applying this Pull Request

grafik

avatar chmst chmst - open - 14 Sep 2021
avatar chmst chmst - change - 14 Sep 2021
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 14 Sep 2021
Category Modules Front End
avatar chmst chmst - change - 14 Sep 2021
Labels Added: ?
avatar ChristineWk ChristineWk - test_item - 14 Sep 2021 - Tested successfully
avatar ChristineWk
ChristineWk - comment - 14 Sep 2021

I have tested this item successfully on 16cdf18

With the newer version it's OK.


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/35560.
avatar ChristineWk
ChristineWk - comment - 14 Sep 2021

Result:
pr35560

avatar brianteeman brianteeman - test_item - 14 Sep 2021 - Tested successfully
avatar brianteeman
brianteeman - comment - 14 Sep 2021

I have tested this item successfully on 16cdf18


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/35560.

avatar alikon alikon - change - 14 Sep 2021
Status Pending Ready to Commit
avatar alikon
alikon - comment - 14 Sep 2021

RTC


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/35560.

avatar wilsonge wilsonge - change - 17 Oct 2021
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2021-10-17 16:50:54
Closed_By wilsonge
Labels Added: ?
avatar wilsonge wilsonge - close - 17 Oct 2021
avatar wilsonge wilsonge - merge - 17 Oct 2021
avatar wilsonge
wilsonge - comment - 17 Oct 2021

Thanks!

Add a Comment

Login with GitHub to post a comment