Bug #30542

Static Route Parts are not reset

Added by Bastian Waidelich almost 4 years ago. Updated almost 4 years ago.

Status:Resolved Start date:2011-10-04
Priority:Must have Due date:
Assigned To:Bastian Waidelich % Done:

100%

Category:MVC - Routing
Target version:TYPO3 Flow Base Distribution - 1.0.0
PHP Version: Complexity:
Has patch: Affected Flow version:

Description

StaticRoutePart::resolve() sets the value to the route parts name (= the corresponding URI part).
RoutePart::match() merges the route parts value into the resulting request arguments. The value of the route part must be reset to NULL before the route parth is processed.
The StaticRoutePart currently fails to do so. Usually this has no effect because match() is always called before resolve() but it fails when using FLOW3 with an app server like AiP.

Associated revisions

Revision 9d03bb52
Added by Bastian Waidelich almost 4 years ago

[BUGFIX] Static Route Parts are not reset

StaticRoutePart::resolve() sets the value to the route parts
name (= the corresponding URI part).
RoutePart::match() merges the route parts value into the
resulting request arguments. The value of the route part must be
reset to NULL before the route parth is processed.
The StaticRoutePart currently fails to do so. Usually this has
no effect because match() is always called before resolve() but
it fails when using FLOW3 with an app server like AiP.

Change-Id: Ide104658cb1c822e58768200092c30dd0b356e33
Resolves: #30542

History

#1 Updated by Bastian Waidelich almost 4 years ago

  • Subject changed from Static Route Parts are not reset before matching URIs to Static Route Parts are not reset

#2 Updated by Mr. Hudson almost 4 years ago

  • Status changed from Accepted to Under Review

Patch set 1 of change Ide104658cb1c822e58768200092c30dd0b356e33 has been pushed to the review server.
It is available at http://review.typo3.org/5528

#3 Updated by Bastian Waidelich almost 4 years ago

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

#4 Updated by Karsten Dambekalns almost 4 years ago

  • Target version changed from 1230 to 1.0.0

Also available in: Atom PDF