Feature RTC bug PR-5.2-dev Pending

User tests: Successful: Unsuccessful:

avatar Hackwar
Hackwar
28 Apr 2024

Summary of Changes

When the route of an entry in Smart Search isn't a query string, the highlight feature actually can produce invalid URLs. This PR enforces these URLs to be correct.

Testing Instructions

Look at #43397, apply that patch and get the testing plugin. Index an RTF file and search in the frontend for this entry in the search. Click on the URL.

Actual result BEFORE applying this Pull Request

The URL looks something like /tolstoy.rtf&highlight=bunchofbase64gibberish and throws a 404 when clicked.

Expected result AFTER applying this Pull Request

The URL looks something like /tolstoy.rtf?highlight=bunchofbase64gibberish and does not throw a 404 when clicked.

Link to documentations

Please select:

  • Documentation link for docs.joomla.org:

  • No documentation changes for docs.joomla.org needed

  • Pull Request link for manual.joomla.org:

  • No documentation changes for manual.joomla.org needed

avatar Hackwar Hackwar - open - 28 Apr 2024
avatar Hackwar Hackwar - change - 28 Apr 2024
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 28 Apr 2024
Category Front End Plugins
avatar Hackwar Hackwar - change - 23 Jul 2024
Labels Added: Feature bug PR-5.2-dev
avatar webnet-assmann webnet-assmann - test_item - 23 Jul 2024 - Tested successfully
avatar webnet-assmann
webnet-assmann - comment - 23 Jul 2024

I have tested this item ✅ successfully on a5bd855


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

avatar viocassel viocassel - test_item - 3 Nov 2024 - Tested successfully
avatar viocassel
viocassel - comment - 3 Nov 2024

I have tested this item ✅ successfully on a5bd855


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

avatar Quy Quy - change - 3 Nov 2024
Status Pending Ready to Commit
avatar Quy
Quy - comment - 3 Nov 2024

RTC


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

avatar Quy Quy - change - 3 Nov 2024
Labels Added: RTC
avatar pe7er pe7er - change - 7 Nov 2024
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2024-11-07 14:21:56
Closed_By pe7er
avatar pe7er pe7er - close - 7 Nov 2024
avatar pe7er pe7er - merge - 7 Nov 2024
avatar pe7er
pe7er - comment - 7 Nov 2024

Thanks @Hackwar !

Add a Comment

Login with GitHub to post a comment