User tests: Successful: Unsuccessful:
The routing in Joomla has had 2 issues in the past: When IDs were enabled in URLs you could basically modify the alias part of the URL at will and if you wanted to switch from ID based URLs to those without, there was no way to automatically redirect everything. This PR tries to fix both issues.
The PR is depending on #44455 to mark a parsed URL as tainted. There are two modes here:
This PR was made possible by the support of djumla. Thank you for that.
When reading the content item or category segment of a URL, the ID is read and based on that ID the alias is read from the database and if the two don't match, the URL is marked as tainted and later redirected to the "correct" URL. (#43992 is fixing the alias for that URL then) This prevents modified aliases in the URL.
If IDs are switched off, we are first searching for the normal content item or category as we are doing now, but when we don't find a match, we assume that we are actually reading a legacy URL with IDs switched on. In that case we mark the URL as tainted (regardless of it being totally broken or not, since when we fail to parse the URL at all, we already throw a 404 before we reach the handling of tainted URLs) and then compare the segment with the segment we would get with IDs enabled. If that matches, we return the ID and later redirect to the correct URL. This means that you can switch from ID-based URLs to "clean" URLs and Joomla will automatically redirect all pages to the correct URLs.
for 1.: You get the same content with more than one URL
for 2.: You get a 404 error
for 1.: You are redirected to the right URL again with the correct alias.
for 2.: You are redirected to the right URL without IDs
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
Status | New | ⇒ | Pending |
Category | ⇒ | Front End com_contact com_content com_newsfeeds |
I have tested this item ✅ successfully on 1e5d91a
Works as described
Labels |
Added:
Feature
PR-5.3-dev
|
Status | Pending | ⇒ | Ready to Commit |
RTC
I have tested this item ✅ successfully on 1e5d91a
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/44477.