Bug #86506

Resolve slashed values in PageTypeDecorator correctly

Added by Oliver Hader about 1 year ago. Updated about 1 year ago.

Status:
Closed
Priority:
Should have
Assignee:
-
Category:
Link Handling, Site Handling & Routing
Target version:
Start date:
2018-10-01
Due date:
% Done:

0%

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

Description

  • slashed values are not resolved correctly concerning it's dynamic '/' prefix in route path
  • adjusting route path and parameters might case side effects when dealing with multiple decorating enhancers
  • test cases for resolving and generating URIs using PageTypeDecorator are added

Associated revisions

Revision 95c085e4 (diff)
Added by Oliver Hader about 1 year ago

[BUGFIX] Resolve slashed values in PageTypeDecorator correctly

  • slashed values are not resolved correctly concerning their
    dynamic '/' prefix in route paths
  • adjusting route path and parameters might cause side effects
    when dealing with multiple decorating enhancers
  • test cases for resolving and generating URIs using new
    PageTypeDecorator enhancer are added

Resolves: #86506
Releases: master
Change-Id: I3757933c47ea0f06d88bb27a635c2077e8f673b1
Reviewed-on: https://review.typo3.org/58512
Reviewed-by: Benni Mack <>
Tested-by: Benni Mack <>
Tested-by: TYPO3com <>
Reviewed-by: Susanne Moog <>
Tested-by: Susanne Moog <>

History

#1 Updated by Gerrit Code Review about 1 year ago

  • Status changed from New to Under Review

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/58512

#2 Updated by Gerrit Code Review about 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/58512

#3 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/58512

#4 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/58512

#5 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/58512

#6 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/58512

#7 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/58512

#8 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/58512

#9 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/58512

#10 Updated by Gerrit Code Review about 1 year ago

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

#11 Updated by Benni Mack about 1 year ago

  • Status changed from Under Review to Closed

Also available in: Atom PDF