Project

General

Profile

Actions

Bug #100990

closed

Pages with trailing slash in slug are not resolved if requested uri does not contain a trailing slash

Added by Stefan Bürk 11 months ago. Updated 10 months ago.

Status:
Resolved
Priority:
Should have
Assignee:
Category:
Link Handling, Site Handling & Routing
Target version:
-
Start date:
2023-06-10
Due date:
% Done:

100%

Estimated time:
TYPO3 Version:
11
PHP Version:
Tags:
routing,page-resolving
Complexity:
Is Regression:
Sprint Focus:

Description

Description

The slug field for pages may contain trailing slash or not.
The generator basically creates it without, but it valid to
enter a trailing slash for a slug manually by the editor.

This led to issues and the past and have been addressed by
changes #86055 and #89091.

At a later point default slug resolving for a language if
a valid localized page exists have been solved, reported
with #88715 and #96010.

If a page slug contains a trailing slash, but the request
does not contain a trailing slash, page is not resolved.

For the mentioned changes above some tests have been added,
sadly not covering all possible constellation and therefore
not detecting this case earlier.

Pages falling into this case should be resolved properly and
covered by a couple of additional tests.

Confirmed for:
- TYPO3 11.5.x
- TYPO3 12.4.x
- TYPO3 main

Not working example

page["slug"]......: /my-page/features/
request...........: /my-page/features

is not resolved. The expectation is, that this page is
resolved correctly.

Working examples

page["slug"]......: /my-page/features/
request...........: /my-page/features/
page["slug"]......: /my-page/features
request...........: /my-page/features/
page["slug"]......: /my-page/features
request...........: /my-page/features

Related issues 5 (1 open4 closed)

Related to TYPO3 Core - Bug #96010: Translated pages can be loaded with slug of default language ClosedStefan Bürk2021-11-17

Actions
Related to TYPO3 Core - Bug #88715: Wrong routes for default url slugs on translated base path pagesClosedStefan Bürk2019-07-10

Actions
Related to TYPO3 Core - Bug #89091: Page with a slug with trailing slash results in 404ClosedStefan Bürk2019-09-05

Actions
Related to TYPO3 Core - Bug #86055: pages.slug always removes the trailing slashClosedBenni Mack2018-08-31

Actions
Related to TYPO3 Core - Bug #96599: Slugs with and without trailing slash are considered different but will confuse URL handling in frontendNeeds FeedbackStefan Bürk2022-01-20

Actions
Actions #1

Updated by Stefan Bürk 11 months ago

  • Related to Bug #96010: Translated pages can be loaded with slug of default language added
  • Related to Bug #88715: Wrong routes for default url slugs on translated base path pages added
  • Related to Bug #89091: Page with a slug with trailing slash results in 404 added
  • Related to Bug #86055: pages.slug always removes the trailing slash added
Actions #2

Updated by Gerrit Code Review 11 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/+/79249

Actions #3

Updated by Gerrit Code Review 11 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/+/79249

Actions #4

Updated by Gerrit Code Review 11 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/+/79249

Actions #5

Updated by Gerrit Code Review 11 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/+/79249

Actions #6

Updated by Gerrit Code Review 11 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/+/79249

Actions #7

Updated by Gerrit Code Review 10 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/+/79249

Actions #8

Updated by Gerrit Code Review 10 months ago

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/+/79679

Actions #9

Updated by Stefan Bürk 10 months ago

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

Updated by Gerrit Code Review 10 months ago

  • Status changed from Resolved to Under Review

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

Actions #11

Updated by Stefan Bürk 10 months ago

  • Status changed from Under Review to Resolved
Actions #12

Updated by Stefan Bürk 10 months ago

  • Related to Bug #96599: Slugs with and without trailing slash are considered different but will confuse URL handling in frontend added
Actions

Also available in: Atom PDF