Project

General

Profile

Actions

Bug #85997

closed

Inconsistent page slugs

Added by Francois Suter over 5 years ago. Updated over 5 years ago.

Status:
Closed
Priority:
Must have
Assignee:
-
Category:
Link Handling, Site Handling & Routing
Target version:
Start date:
2018-08-27
Due date:
% Done:

100%

Estimated time:
TYPO3 Version:
9
PHP Version:
7.2
Tags:
Complexity:
Is Regression:
Sprint Focus:

Description

Consider the following:

  • create a page called "First page" at the site root => its slug is "/first-page"
  • rename the page to "First nice page", but don't change the slug
  • create a sub-page called "First sub-page" => its slug is "/first-nice-page/first-sub-page"

Now if you access something like www.foo.com/first-nice-page/first-sub-page in a browser, you might be tempted to remove the last segment in the address bar to access the parent page. However www.foo.com/first-nice-page does not exist and will result in a page not found error.

Actions #1

Updated by Francois Suter over 5 years ago

I'm sure the topic has been given a lot of thought, but is it really a good idea that the slug of a given page contains a whole path to the page, including the slugs of all parent pages? That makes really hard to keep consistency when pages are renamed.

Actions #2

Updated by Mathias Schreiber over 5 years ago

This part is current work in progress.
So here's what will happen:
Once you rename the page, TYPO3 will ask the user what to do with subsequent changes.
In your case this is either:
  • keep slug as is (take the behavior you described)
  • change this slug (and child slugs, take SEO penalty)
  • change slug, create redirects

To address your last question in the comment:
Having only parts of the slug in a page is exactly what realURL does and that's guessing.
Guessing results in problems.

I completely agree that the slug generator should create slugs based on parent slugs (in your case the subpage would've gotten foo.com/first-page/first-sub-page as a slug).
I strongly disagree on only storing parts of the path on page level :)

Actions #3

Updated by Gerrit Code Review over 5 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/58100

Actions #4

Updated by Gerrit Code Review over 5 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/58100

Actions #5

Updated by Benni Mack over 5 years ago

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

Updated by Benni Mack over 5 years ago

  • Status changed from Resolved to Closed
Actions

Also available in: Atom PDF