Project

General

Profile

Actions

Feature #87085

closed

Slug generator option fields should respect list of field definitions as fallback (nav_title // title)

Added by Joerg Boesche almost 6 years ago. Updated over 5 years ago.

Status:
Closed
Priority:
Must have
Assignee:
-
Category:
Site Handling, Site Sets & Routing
Target version:
Start date:
2018-12-05
Due date:
% Done:

100%

Estimated time:
PHP Version:
7.2
Tags:
routing, pages
Complexity:
Sprint Focus:

Description

The slug generator option still uses the field "title" to generate the slug. It is not possible to define a list of fields "nav_title // title", as we know from TypoScript.

In case of SEO optimizations and the daily work of an editor, it should be possible to define a list of fields in the slug configuration.
An editor creates a new page and fills up the title and nav title. Actually the title is used to generate the slug. The behavior should be configurable with the "TCA columnsOverrides" mechanism.

TCA Example

'columns' => [
                'slug' => [
                    'config' => [
                        'generatorOptions' => [
                            'fields' => ['nav_title // title']
                        ]
                    ]
                ],

Example with defined nav_title:
Page-Title: Best summer deals
Navigation Title: Summer deals
Slug: /summer-deals (nav_title)
Slug-behavior: nav_title (is not empty) // title

Example with undefined nav_title, but title:
Page-Title: Best summer deals
Navigation Title: Summer deals
Slug: /best-summer-deals (title)
Slug-behavior: nav_title (is empty, fallback to title) // title


Related issues 1 (0 open1 closed)

Related to TYPO3 Core - Bug #87828: Wrong test case for SlugHelperTestClosedFrank Nägler2019-03-03

Actions
Actions #1

Updated by Joerg Boesche almost 6 years ago

Additional information:
Handling in old realurl extension for pathsegment generation:

'segTitleFieldList' => 'alias,tx_realurl_pathsegment,nav_title,title'

Actions #2

Updated by Gerrit Code Review almost 6 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/59167

Actions #3

Updated by Gerrit Code Review almost 6 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/59167

Actions #4

Updated by Guido Schmechel almost 6 years ago

Using of // is a good direction. Integrators know this convention. Already now you can specify a comma separated list, see RST file. But this simply combines the fields.

Actions #5

Updated by Gerrit Code Review almost 6 years 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/59167

Actions #6

Updated by Gerrit Code Review almost 6 years ago

Patch set 4 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/59167

Actions #7

Updated by Gerrit Code Review almost 6 years ago

Patch set 5 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/59167

Actions #8

Updated by Gerrit Code Review almost 6 years ago

Patch set 6 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/59167

Actions #9

Updated by Gerrit Code Review almost 6 years ago

Patch set 7 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/59167

Actions #10

Updated by Gerrit Code Review almost 6 years ago

Patch set 8 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/59167

Actions #11

Updated by Gerrit Code Review almost 6 years ago

Patch set 9 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/59167

Actions #12

Updated by Gerrit Code Review almost 6 years ago

Patch set 10 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/59167

Actions #13

Updated by Gerrit Code Review almost 6 years ago

Patch set 11 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/59167

Actions #14

Updated by Gerrit Code Review over 5 years ago

Patch set 12 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/+/59167

Actions #15

Updated by Gerrit Code Review over 5 years ago

Patch set 13 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/+/59167

Actions #16

Updated by Gerrit Code Review over 5 years ago

Patch set 1 for branch 9.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/+/59828

Actions #17

Updated by Guido Schmechel over 5 years ago

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

Updated by Frank Nägler over 5 years ago

  • Related to Bug #87828: Wrong test case for SlugHelperTest added
Actions #19

Updated by Gerrit Code Review over 5 years ago

  • Status changed from Resolved to Under Review

Patch set 2 for branch 9.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/+/59828

Actions #20

Updated by Guido Schmechel over 5 years ago

  • Status changed from Under Review to Resolved
Actions #21

Updated by Benni Mack over 5 years ago

  • Status changed from Resolved to Closed
Actions

Also available in: Atom PDF