Feature #106363
closedCheck for redirects with dynamic URLs
100%
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?
Updated by Philipp Kitzberger 3 months ago
- Related to Feature #98567: Check for redirects without existing target page added
Updated by Gerrit Code Review 3 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/+/88604
Updated by Gerrit Code Review 3 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/+/88604
Updated by Gerrit Code Review 3 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/+/88604
Updated by Gerrit Code Review 3 months 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
Updated by Gerrit Code Review 3 months ago
Patch set 5 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
Updated by Gerrit Code Review 3 months ago
Patch set 6 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
Updated by Philipp Kitzberger 3 months ago
- Status changed from Under Review to Resolved
- % Done changed from 0 to 100
Applied in changeset 587fa2f4f96966e074e1909d93fc226005282dbe.
Updated by Garvin Hicking 3 months ago
- Related to Bug #106442: Fix Changelog rendering errors: wrong redirects link, wrong typoscript codeblock added