Bug #99624
closedSuppress sys_redirects generation for storage folders (sysfolders)
100%
Description
Problem/Description¶
Found in TYPO3 11.5.stable:
While working on a "soft"-relaunch I updated a bunch of pages in the page tree. I've used the automatic redirect generation on purpose. Unfortunately also redirects for storage folders were created.
Why got sys folders respected?
Maybe b13/masi
has some missing features here and was the main cause for a redirect on a storage folder?! Because of the amount of pages in multiple site page tree's this extension helps a lot. I also had some similar scenarios without "slug-helper-extensions" so far.
Real World Example "News Storage folder":¶
Before moving pages around
After moving pages (of any kind) around
This leads to following redirect:
domain.tld/news → domain.tld/storage/news
.
What major drawback does here drops in?¶
Possibility of deleting wrong redirect probably prevented¶
No self-service'sh solution if an limited backend editor has no access to the redirects backend module (even if the person has advanced editor skills)
No table listing in access configuration for redirects¶
For better UX, the sys_redirect tables must not be shown for every backend user. (the sys redirect was created right in the sysfolder /storage/news
A major problem which should be covered: Create a new default page called "News" right under the root page¶
- every request on
domain.tld/news
gets redirected todomain.tld/storage/news
which leads – of course – in a 404 - the editor (also a technically savvy one) gets no info about the hidden gem (redirect)
- I can't give any project manager, customer the responsibility for checking the response headers for
x-redirect-by
- I can't give any project manager, customer the responsibility for checking the response headers for
General problem on the blue notifications after moving pages¶
The blue notification is somehow hard to understand. I've heard that from many TYPO3 editors already. Maybe one solution is to disable the automatic redirection and instruct editors to create redirects manually – maybe also a good way to make the topic transparent? Who knows...
ToDo(s)¶
Introduce generic "helpers" in the TYPO3 backend to inform every editor about slug-redirect mismatching.
Or do you have some other advices on this topic? Do we have some missing advices in the TYPO3 documentation about the redirect-topic?
I had already some hours invested to discuss some possibilities in the team. But didn't found any working solution for editors.
Files
Updated by Stefan Bürk over 1 year ago
- Related to Task #101226: Improve the auto-redirect notification on slug change added
Updated by Stefan Bürk over 1 year ago
Created dedicated issues:
- "Blue Notification" -> #101226
Regartding the "permission" issue and additional visual overwork is already on the todo list, albeit not
a real solution yet. Did not turned out for v12, hopefully for v13 then.
However, I would narrow down this issue to the "SysFolder" slug change issue, and additional take "Spacer"
the same way. To be complete, sub-pages which are not sysfolder or spacer should get the slug updated if
it starts with the original one of the sysfolder - like it is for normal page tree traversal.
Updated by Gerrit Code Review over 1 year ago
- Status changed from New to Under Review
Patch set 1 for branch main of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/79623
Updated by Gerrit Code Review over 1 year ago
Patch set 2 for branch main of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/79623
Updated by Gerrit Code Review over 1 year ago
Patch set 3 for branch main of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/79623
Updated by Stefan Bürk over 1 year ago
- Status changed from Under Review to Resolved
- % Done changed from 0 to 100
Applied in changeset e9131cd20f5d4ca8db1be81b31116044cc0b2e59.
Updated by Gerrit Code Review over 1 year ago
- Status changed from Resolved to Under Review
Patch set 1 for branch 12.4 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/79695
Updated by Gerrit Code Review over 1 year ago
Patch set 2 for branch 12.4 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/79695
Updated by Gerrit Code Review over 1 year ago
Patch set 3 for branch 12.4 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/79695
Updated by Stefan Bürk over 1 year ago
- Status changed from Under Review to Resolved
Applied in changeset 1a3cf896ec782f835c86c357da796d83f1a23a78.
Updated by RVVN no-lastname-given over 1 year ago
Hi,
I know that codebase differs from v12 for redirect handling but is there any chance that this issue would be fixed on TYPO3v11 ?