Bug #88291

Exception thrown if slash in route field of PersistedPatternMapper aspect

Added by Daniel Reichel about 2 years ago. Updated about 1 year ago.

Status:
Closed
Priority:
Must have
Assignee:
-
Category:
Link Handling, Site Handling & Routing
Target version:
-
Start date:
2019-05-07
Due date:
% Done:

100%

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

Description

In my site configuration I have configured a aspect of type PersistedPatternMapper for the detail view of a record of my custom extension.
If the configured route field contains a slash, a error occurs in the frontend.

Exception thrown in frontend

You can reproduce the bug using the news extension. I know, the news extension has a slug field but my custom extension which you dont't have has not.
This is the site configuration:

routeEnhancers:
  NewsPlugin:
    type: Extbase
    extension: News
    plugin: Pi1
    routes:
      - { routePath: '/news/{news}', _controller: 'News::detail', _arguments: {'news': 'news_title'} }
    defaultController: 'News::detail'
    aspects:
      news:
        type: PersistedPatternMapper
        tableName: 'tx_news_domain_model_news'
        routeFieldPattern: '^(?P<title>.+)-(?P<uid>\d+)$'
        routeFieldResult: '{title}-{uid}'

Files

exception.jpg (137 KB) exception.jpg Exception thrown in frontend Daniel Reichel, 2019-05-07 13:40

Related issues

Related to TYPO3 Core - Bug #86596: Slash in Page Title still in URLClosed2018-10-08

Actions
Related to TYPO3 Core - Bug #87667: Replacements not applied for existing slugNeeds Feedback2019-02-06

Actions
Related to TYPO3 Core - Bug #87656: Slugs error if title has "/" in extbase extensionClosed2019-02-05

Actions
Related to TYPO3 Core - Bug #87333: Slug fieldSeparator default value collides with symfony strict requirementsClosed2019-01-05

Actions
Related to TYPO3 Core - Bug #89263: TCEMAIN.preview doesn't work for routed controller with empty argumentsNew2019-09-25

Actions
Related to TYPO3 Core - Bug #90531: Requirements are not considered when an aspect is presentClosed2020-02-25

Actions
Related to TYPO3 Core - Feature #90814: Allow slash in routeFieldResult and routeFieldPattern to allow optional pathsClosed2020-03-23

Actions
#1

Updated by Daniel Reichel about 2 years ago

  • Related to Bug #86596: Slash in Page Title still in URL added
#2

Updated by Daniel Reichel about 2 years ago

  • Related to Bug #87667: Replacements not applied for existing slug added
#3

Updated by Daniel Reichel about 2 years ago

  • Related to Bug #87656: Slugs error if title has "/" in extbase extension added
#4

Updated by Bastian Bringenberg about 2 years ago

  • Project changed from forge.typo3.org to TYPO3 Core
  • TYPO3 Version set to 9

Moved Project.

#5

Updated by Riccardo De Contardi about 2 years ago

  • Category set to Link Handling, Site Handling & Routing
#6

Updated by Christian Eßl over 1 year ago

  • Related to Bug #87333: Slug fieldSeparator default value collides with symfony strict requirements added
#7

Updated by Christian Eßl over 1 year ago

  • Related to Bug #89263: TCEMAIN.preview doesn't work for routed controller with empty arguments added
#8

Updated by Christian Eßl over 1 year ago

  • Related to Bug #90531: Requirements are not considered when an aspect is present added
#9

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/c/Packages/TYPO3.CMS/+/63529

#10

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/c/Packages/TYPO3.CMS/+/63529

#11

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/c/Packages/TYPO3.CMS/+/63529

#12

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/c/Packages/TYPO3.CMS/+/63529

#13

Updated by Gerrit Code Review over 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/+/63655

#14

Updated by Christian Eßl over 1 year ago

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

Updated by Patrick Crausaz about 1 year ago

  • Related to Feature #90814: Allow slash in routeFieldResult and routeFieldPattern to allow optional paths added
#16

Updated by Benni Mack about 1 year ago

  • Status changed from Resolved to Closed

Also available in: Atom PDF