Project

General

Profile

Actions

Bug #99624

closed

Suppress sys_redirects generation for storage folders (sysfolders)

Added by Josef Glatz over 1 year ago. Updated 10 months ago.

Status:
Resolved
Priority:
Could have
Assignee:
-
Category:
Redirect Handling
Target version:
-
Start date:
2023-01-19
Due date:
% Done:

100%

Estimated time:
TYPO3 Version:
12
PHP Version:
Tags:
Complexity:
nightmare
Is Regression:
Sprint Focus:

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 to domain.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

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


Related issues 1 (1 open0 closed)

Related to TYPO3 Core - Task #101226: Improve the auto-redirect notification on slug changeNewStefan Bürk2023-07-03

Actions
Actions #1

Updated by Stefan Bürk 10 months ago

  • Related to Task #101226: Improve the auto-redirect notification on slug change added
Actions #2

Updated by Stefan Bürk 10 months ago

Created dedicated issues:

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.

Actions #3

Updated by Gerrit Code Review 10 months 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

Actions #4

Updated by Gerrit Code Review 10 months 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

Actions #5

Updated by Gerrit Code Review 10 months 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

Actions #6

Updated by Stefan Bürk 10 months ago

  • Status changed from Under Review to Resolved
  • % Done changed from 0 to 100
Actions #7

Updated by Gerrit Code Review 10 months 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

Actions #8

Updated by Gerrit Code Review 10 months 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

Actions #9

Updated by Gerrit Code Review 10 months 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

Actions #10

Updated by Stefan Bürk 10 months ago

  • Status changed from Under Review to Resolved
Actions #11

Updated by RVVN no-lastname-given 10 months 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 ?

Actions

Also available in: Atom PDF