Feature #91776: Use the pid field to associate a redirect with the page it was created for - TYPO3 Core - TYPO3 Forge

Actions

Copy link

Feature #91776

closed

Use the pid field to associate a redirect with the page it was created for

Added by Oliver Eglseder over 4 years ago. Updated about 3 years ago.

Status:

Closed

Priority:

Should have

Assignee:

Category:

Site Handling, Site Sets & Routing

Target version:

next-patchlevel

Start date:

2020-07-09

Due date:

% Done:

100%

Estimated time:

PHP Version:

Tags:

site, sys_redirect

Complexity:

no-brainer

Sprint Focus:

Description

I have a task which requires me to associate generated redirects (through slug changes) with the page that triggered the generation.
Example:

I have the page tree

New TYPO3 site
 `- [1] Root Page     (slug "/")
     |- [2] Something (slug "/somehting")
     `- [3] Else      (slug "/else")

I change the slug of "[1] Root Page" to "start", which updates all slugs recursively

New TYPO3 site
 `- [1] Root Page     (slug "/start")
     |- [2] Something (slug "/start/somehting")
     `- [3] Else      (slug "/start/else")

This action will create 3 redirects

uid	pid	source_host	source_path	target
1	0	*	/	/start
2	0	*	/somehting	/start/somehting
3	0	*	/else	/start/else

I would like to have these redirects associated with the page that got changed and triggered the creation of these redirects.

uid	pid	source_host	source_path	target
1	1	*	/	/start
2	1	*	/somehting	/start/somehting
3	1	*	/else	/start/else

changing the slug of the page "[2] Something" to "something" (correcting the typo) would therefore create a redirect with PID 2

uid pid source_host source_path target

4 2 * /start/somehting /start/something

Implications:

In perspective this could be also used to show editors all redirect sources to a page (a list of all URLs that are redirected to the selected page)
Should be fully backwards compatible. I did not find an obvious reason to keep them all on pid 0.
It makes it easier for EXT:in2publish_core to find redirects for pages and publish pages including their redirects.

Related issues 4 (0 open — 4 closed)

Actions

Copy link

Updated by Gerrit Code Review over 3 years ago

Status changed from New to Under Review

Patch set 1 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/70597

Actions

Copy link

Updated by Gerrit Code Review over 3 years ago

Patch set 2 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/70597

Actions

Copy link

Updated by Gerrit Code Review about 3 years ago

Patch set 3 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/70597

Actions

Copy link