Feature #87085

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

Added by Joerg Boesche over 1 year ago. Updated 11 months ago.

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

100%

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

Related to TYPO3 Core - Bug #87828: Wrong test case for SlugHelperTest Closed 2019-03-03

Associated revisions

Revision ec60f914 (diff)
Added by Guido Schmechel about 1 year ago

[FEATURE] Fallback options for slug fields

Now it is possible to define several fields with fallback function for
the generatorOptions fields.

Releases: master, 9.5
Resolves: #87085
Change-Id: I985f35267955dc43baaad2263ed8947c2c87b7c6
Reviewed-on: https://review.typo3.org/c/59167
Reviewed-by: Martin Kutschker <>
Reviewed-by: André Schließer <>
Reviewed-by: Sebastian Rosskopf <>
Reviewed-by: Anja Leichsenring <>
Tested-by: André Schließer <>
Tested-by: Sebastian Rosskopf <>
Tested-by: TYPO3com <>
Tested-by: Anja Leichsenring <>

Revision 801a2162 (diff)
Added by Frank Naegler about 1 year ago

[BUGFIX] Fix wrong test case

With #87085 a wrong test case was introduced. this patch fix the wrong
test case.

Resolves: #87828
Related: #87085
Releases: master
Change-Id: I79b44bf8739f2869f32fbc85d85d92d2feea9987
Reviewed-on: https://review.typo3.org/c/59836
Tested-by: TYPO3com <>
Tested-by: Richard Haeser <>
Tested-by: Andreas Fernandez <>
Tested-by: Daniel Goerz <>
Tested-by: Georg Ringer <>
Reviewed-by: Richard Haeser <>
Reviewed-by: Andreas Fernandez <>
Reviewed-by: Daniel Goerz <>
Reviewed-by: Georg Ringer <>

Revision 6fcf613d (diff)
Added by Guido Schmechel about 1 year ago

[FEATURE] Fallback options for slug fields

Now it is possible to define several fields with fallback function for
the generatorOptions fields.

Releases: master, 9.5
Resolves: #87085
Change-Id: I985f35267955dc43baaad2263ed8947c2c87b7c6
Reviewed-on: https://review.typo3.org/c/59828
Tested-by: TYPO3com <>
Tested-by: Anja Leichsenring <>
Tested-by: Frank Naegler <>
Reviewed-by: Anja Leichsenring <>
Reviewed-by: Frank Naegler <>

History

#1 Updated by Joerg Boesche over 1 year ago

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

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

#2 Updated by Gerrit Code Review over 1 year 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

#3 Updated by Gerrit Code Review over 1 year 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

#4 Updated by Guido Schmechel over 1 year 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.

#5 Updated by Gerrit Code Review over 1 year 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

#6 Updated by Gerrit Code Review over 1 year 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

#7 Updated by Gerrit Code Review over 1 year 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

#8 Updated by Gerrit Code Review over 1 year 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

#9 Updated by Gerrit Code Review about 1 year 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

#10 Updated by Gerrit Code Review about 1 year 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

#11 Updated by Gerrit Code Review about 1 year 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

#12 Updated by Gerrit Code Review about 1 year 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

#13 Updated by Gerrit Code Review about 1 year 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

#14 Updated by Gerrit Code Review about 1 year 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

#15 Updated by Gerrit Code Review about 1 year 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

#16 Updated by Gerrit Code Review about 1 year 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

#17 Updated by Guido Schmechel about 1 year ago

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

#18 Updated by Frank Naegler about 1 year ago

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

#19 Updated by Gerrit Code Review about 1 year 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

#20 Updated by Guido Schmechel about 1 year ago

  • Status changed from Under Review to Resolved

#21 Updated by Benni Mack 11 months ago

  • Status changed from Resolved to Closed

Also available in: Atom PDF