RTC Release Blocker bug PR-5.1-dev Pending

User tests: Successful: Unsuccessful:

avatar Hackwar
Hackwar
12 Mar 2024

Pull Request for Issue #42991.

Summary of Changes

In an ideal world, the site router would only be created when necessary with the previously added onAfterInitialiseRouter event. However if a system plugin requests the site router object before all constructors of system plugins have run, the event would be fired, but no plugin would react to it, since the events haven't been registered yet. Since we can't guarantee that no one requests the router object prematurely, we have to revert the code and load the site router regardless of the application executed.

Testing Instructions

  1. Edit plugins/system/redirect/services/provider.php and insert $container->get(\Joomla\CMS\Router\SiteRouter::class); at line 43 (after '$plugin->setDatabase($container->get(DatabaseInterface::class));')
  2. Make sure the languagefilter plugin is enabled
  3. Open any page

Actual result BEFORE applying this Pull Request

The page throws an error.

Expected result AFTER applying this Pull Request

The page works fine.

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 - 12 Mar 2024
avatar Hackwar Hackwar - change - 12 Mar 2024
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 12 Mar 2024
Category Libraries Front End Plugins
avatar Fedik
Fedik - comment - 12 Mar 2024

Please wait with it, I need time to check different approach.

avatar SniperSister SniperSister - test_item - 13 Mar 2024 - Tested successfully
avatar SniperSister
SniperSister - comment - 13 Mar 2024

I have tested this item ✅ successfully on 479dcac

Works as described. For more info see my test comments in #43015


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

avatar richard67
richard67 - comment - 13 Mar 2024

@Hackwar Does this close issue #42991 ?

avatar Hackwar
Hackwar - comment - 13 Mar 2024

yes

avatar Fedik Fedik - test_item - 14 Mar 2024 - Tested successfully
avatar Fedik
Fedik - comment - 14 Mar 2024

I have tested this item ✅ successfully on 479dcac


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

avatar Fedik Fedik - change - 14 Mar 2024
Status Pending Ready to Commit
avatar Fedik
Fedik - comment - 14 Mar 2024

r2c


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

avatar LadySolveig LadySolveig - change - 14 Mar 2024
Labels Added: RTC Release Blocker bug PR-5.1-dev
avatar LadySolveig LadySolveig - change - 14 Mar 2024
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2024-03-14 17:29:58
Closed_By LadySolveig
avatar LadySolveig LadySolveig - close - 14 Mar 2024
avatar LadySolveig LadySolveig - merge - 14 Mar 2024
avatar LadySolveig
LadySolveig - comment - 14 Mar 2024

Thank you @Hackwar and for the support and testing @Fedik @SniperSister

Add a Comment

Login with GitHub to post a comment