Feature #91776

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

Added by Oliver Eglseder over 1 year ago. Updated about 2 months ago.

Status:
Closed
Priority:
Should have
Assignee:
-
Category:
Link Handling, Site Handling & Routing
Target version:
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.
#1

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

#2

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

#3

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

#4

Updated by Stefan Busemann about 2 months ago

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

Updated by Benni Mack about 2 months ago

  • Status changed from Resolved to Closed

Also available in: Atom PDF