Bug #33519

Route always appends a question mark even if the route already has arguments

Added by Aske Ertmann over 3 years ago. Updated about 3 years ago.

Status:Resolved Start date:2012-01-29
Priority:Must have Due date:
Assigned To:Aske Ertmann % Done:

100%

Category:MVC - Routing
Target version:TYPO3 Flow Base Distribution - 1.1
PHP Version:5.3 Complexity:no-brainer
Has patch:Yes Affected Flow version:Git master

Description

The Route class (MVC\Web\Routing\Route) always append additionalParameters etc., with a question mark in front even though the url could have arguments already. This happens if a routehandler resolves everything & creates the argument string by it self.


Related issues

related to TYPO3.Flow - Feature #33539: Append query arguments from RoutePart handlers Rejected 2012-01-30
duplicated by TYPO3.Flow - Bug #36859: URLs Break if the Route return a URI with Query string Closed 2012-05-05

Associated revisions

Revision dc3532f4
Added by Aske Ertmann over 3 years ago

[BUGFIX] Prevent Route class from appending an extra question mark

Change-Id: I965367962786ab1ba923dbaae304e0d02afdfdc1
Fixes: #33519
Related: #33539
Releases: 1.1

History

#1 Updated by Gerrit Code Review over 3 years ago

  • Status changed from New to Under Review

Patch set 1 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/8766

#2 Updated by Gerrit Code Review over 3 years ago

Patch set 2 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/8766

#3 Updated by Gerrit Code Review over 3 years ago

Patch set 3 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/8766

#4 Updated by Gerrit Code Review over 3 years ago

Patch set 4 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/8766

#5 Updated by Gerrit Code Review over 3 years ago

Patch set 5 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/8766

#6 Updated by Karsten Dambekalns about 3 years ago

  • Status changed from Under Review to Resolved
  • Assigned To set to Aske Ertmann
  • % Done changed from 0 to 100

Also available in: Atom PDF