[6.0] Languagefilter: Refactoring of routing

[Hackwar]

Pull Request for issue #24315, #25449, #39181

Summary of Changes

This is a refactoring of the routing component of the languagefilter plugin. The goal are as follows:

• Allow to parse URLs more than once. Right now the parseing of the languagefilter plugin sets a lot of state and even does redirects.
• When the prefix of the default language is set to be removed, the redirect for the language switch is removed.
• Make the code simpler and more reliable.

Right now this PR is a Proof Of Concept and is NOT ready. This is to collect feedback from the community and other developers.

Testing Instructions

Setup different multilanguage sites and apply the patch. The default behavior should not change, except that when the default prefix is removed, you can switch the language without an additional redirect.

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

[Slava287]

Works. Thank you!

[brianteeman]

@Slava287 Please go to the issue tracker here https://issues.joomla.org/tracker/joomla-cms/43858 and mark your test result by using the blue "Test this" button at the top left corner, selecting your test result and finally submit, so that your test is properly counted. Just a comment here is not enough. Thanks in advance.

[Slava287]

I have tested this item ✅ successfully on 3f9a0c5

Everything works correctly. There are no errors. I also tested it on the latest version of Joomla! 5.1.3 Stable

---

_{This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/43858.}

[Slava287]

Ok

---

_{This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/43858.}

[Hackwar]

When suffix are enabled, this unfortunately right now creates URLs like /en.html. I still have to take a look what we have to do here.

[Slava287]

Hello. Indeed, I didn’t notice it right away

[Hackwar]

Now the URLs should all be generated correctly.

[exlemor]

I have tested this item 🔴 unsuccessfully on 22e4bd1

I tested this unsuccessfully on Joomla ‎6.0.0-alpha2-dev (Nightly from June 17th) and sadly it crashes as soon as I turn the patch on with:

0 Call to a member function getInput() on null

Sorry, :( @Hackwar

---

_{This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/43858.}

[Hackwar]

Seems like this was a merge conflict at some point. I fixed that. Thanks @exlemor! If you had time, could you test this again? @Slava287 if you have time to test this again as well, I would be very happy. Thank you!

[exlemor]

I have tested this item ✅ successfully on 99eb3e8

I have successfully tested this. Added language, removed it, checked articles associations, etc... Checked Front-end/Back-end.

---

_{This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/43858.}

[brianteeman]

Right now this PR is a Proof Of Concept and is NOT ready.

Is that still the case?

[Hackwar]

Nope, this PR is ready.

[brianteeman]

Can you update the post then - otherwise people wont test it

[richard67]

Can you update the post then - otherwise people wont test it

@Hackwar I think it should be sufficient to remove the following 2 sentences from the description of this PR:

Right now this PR is a Proof Of Concept and is NOT ready. This is to collect feedback from the community and other developers.

[sdwjoomla]

I have tested this item ✅ successfully on 5fb4733

Tested with French language pack

---

_{This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/43858.}

[richard67]

RTC

---

_{This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/43858.}

[Bodge-IT]

Thanks @Hackwar for the change and thanks @exlemor & @sdwjoomla for tests.