Project

General

Profile

Actions

Feature #106363

open

Check for redirects with dynamic URLs

Added by Philipp Kitzberger 12 days ago. Updated 8 days ago.

Status:
Under Review
Priority:
Should have
Assignee:
-
Category:
Link Handling & Redirect Handling
Target version:
-
Start date:
2025-03-12
Due date:
% Done:

0%

Estimated time:
PHP Version:
Tags:
Complexity:
Sprint Focus:

Description

Currently the redirect integrity check is only checking slugs of records of the pages table. That obviously ignores any dynamically rendered URLs such as news single views. These URLs should be checked as well, when generating the report of conflicting slugs/redirects.

The obvious improvement would be to dispatch an event so other extensions could provide their URLs before they're all checked against the redirects.

Or simply change the way IntegrityService->getAllPageUrlsForSite() is fetching URLs: instead of a DB lookup into pages we could simple look into the URLs rendered in the XML sitemap. No?


Related issues 1 (1 open0 closed)

Related to TYPO3 Core - Feature #98567: Check for redirects without existing target pageNew2022-10-10

Actions
Actions #1

Updated by Philipp Kitzberger 12 days ago

  • Related to Feature #98567: Check for redirects without existing target page added
Actions #2

Updated by Gerrit Code Review 8 days 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/+/88604

Actions #3

Updated by Gerrit Code Review 8 days 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/+/88604

Actions #4

Updated by Gerrit Code Review 8 days 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/+/88604

Actions #5

Updated by Gerrit Code Review 8 days ago

Patch set 4 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/+/88604

Actions

Also available in: Atom PDF