Feature RTC PR-5.3-dev Pending

User tests: Successful: Unsuccessful:

avatar joomdonation
joomdonation
8 Feb 2025

Pull Request for Issue # .

Summary of Changes

This PR improves System - Language Filter code:

  • Add missing $event parameter to event listener methods. Also add void return type declaration to these methods (all event listener methods should have void return type)
  • Introduce $app and $language variables to avoid repeat calling methods to get these objects
  • Fixed a small bug with onPrivacyCollectAdminCapabilities does not return data properly. It needs to use $event->addResult to return data instead of return command. I will backport this bug fix to 5.2 if required

Testing Instructions

  • Expect to have at least one code review from maintainer
  • Setup a multilingual website using Joomla 5.3 nightly build, apply patch, navigate to random pages from frontend of your site and make sure nothing is broken.

Actual result BEFORE applying this Pull Request

Works

Expected result AFTER applying this Pull Request

Works, code is better. A small bug is fixed.

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 joomdonation joomdonation - open - 8 Feb 2025
avatar joomdonation joomdonation - change - 8 Feb 2025
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 8 Feb 2025
Category Front End Plugins
f54e47a 8 Feb 2025 avatar joomdonation CS
avatar joomdonation joomdonation - change - 8 Feb 2025
Labels Added: PR-5.3-dev
avatar Fedik Fedik - test_item - 9 Feb 2025 - Tested successfully
avatar Fedik
Fedik - comment - 9 Feb 2025

I have tested this item ✅ successfully on f54e47a


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

avatar richard67
richard67 - comment - 9 Feb 2025

Hmm, with this PR applied, all was working during my tests, but I got a PHP warning which I do not get without this PR applied:

PHP Warning:  Attempt to read property "sef" on null in /home/richard/lamp/public_html/joomla-cms-5.3-dev/plugins/system/languagefilter/src/Extension/LanguageFilter.php on line 278

I get this when using the button in the backend for going to the frontend or when using the site URL without any language suffix appended, i.e. the pure domain only.

Update: And I get it only when having this PR here applied and in addition PR #44844 for the debug plugin. Weird.

Update: Ot happens also when I apply only PR #44844 but not this one here. Will report in PR #44844 .

avatar richard67 richard67 - test_item - 9 Feb 2025 - Tested successfully
avatar richard67
richard67 - comment - 9 Feb 2025

I have tested this item ✅ successfully on f54e47a


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

avatar richard67 richard67 - change - 9 Feb 2025
Status Pending Ready to Commit
avatar richard67
richard67 - comment - 9 Feb 2025

RTC


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

avatar laoneo laoneo - change - 9 Feb 2025
Labels Added: Feature RTC
avatar laoneo laoneo - change - 9 Feb 2025
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2025-02-09 17:15:14
Closed_By laoneo
avatar laoneo laoneo - close - 9 Feb 2025
avatar laoneo laoneo - merge - 9 Feb 2025

Add a Comment

Login with GitHub to post a comment