RTC PR-4.4-dev Pending

User tests: Successful: 0 Unsuccessful: 0

avatar SniperSister
SniperSister
27 Aug 2024

Summary of Changes

So far com_finder taxonomy rows had an empty string as default parameter for the language column. Taxonomy rows with that empty string became unselectable one the multilanguage feature was enabled, as the language filter only filtered for the current language and the * character, see:
https://github.com/joomla/joomla-cms/blob/4.4-dev/administrator/components/com_finder/src/Service/HTML/Filter.php#L306

Testing Instructions

  • Run the finder indexer, view the #__finder_taxonomy table and check the taxonomy column values.
  • Apply the patch.
  • Reindex
  • Check the DB table again

Actual result BEFORE applying this Pull Request

Rows with empty string.

Expected result AFTER applying this Pull Request

Rows with empty strings are now using * as value

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 SniperSister SniperSister - open - 27 Aug 2024
avatar SniperSister SniperSister - change - 27 Aug 2024
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 27 Aug 2024
Category Administration com_finder
avatar fgsw fgsw - test_item - 28 Aug 2024 - Tested successfully
avatar fgsw
fgsw - comment - 28 Aug 2024

I have tested this item ✅ successfully on c61e65b

Without Patch:

patch-no

With Patch:

patch


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/43990.
avatar SniperSister SniperSister - change - 28 Aug 2024
Labels Added: PR-4.4-dev
avatar SniperSister
SniperSister - comment - 28 Aug 2024

@fgsw thanks for testing, that's very much appreciated! I have pushed another commit that will also fix the language in the first taxonomy levels that you marked red in your screenshot. Could you please re-rest? THANK YOU!

avatar fgsw fgsw - test_item - 28 Aug 2024 - Tested successfully
avatar fgsw
fgsw - comment - 28 Aug 2024

I have tested this item ✅ successfully on 0120dfd


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

avatar Hackwar Hackwar - change - 28 Aug 2024
The description was changed
avatar Hackwar Hackwar - edited - 28 Aug 2024
avatar ChristineWk
ChristineWk - comment - 29 Aug 2024

@fgsw Thank you for screenshots for my better understanding :-)


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

avatar ChristineWk ChristineWk - test_item - 29 Aug 2024 - Tested successfully
avatar ChristineWk
ChristineWk - comment - 29 Aug 2024

I have tested this item ✅ successfully on 0120dfd


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

avatar Quy Quy - change - 29 Aug 2024
Status Pending Ready to Commit
avatar Quy
Quy - comment - 29 Aug 2024

RTC


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

avatar Quy Quy - change - 29 Aug 2024
Labels Added: RTC
avatar laoneo laoneo - change - 30 Aug 2024
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2024-08-30 06:22:27
Closed_By laoneo
avatar laoneo laoneo - close - 30 Aug 2024
avatar laoneo laoneo - merge - 30 Aug 2024

Add a Comment

Login with GitHub to post a comment