adulau/cakephp2-php8
1
0
Fork 0
mirror of https://github.com/kamilwylegala/cakephp2-php8.git synced 2024-11-15 03:18:26 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions

Adding support for a trailing & or & in the existing queryString

Browse source
This commit is contained in:
Jelle Henkens 2012-05-09 22:32:57 +01:00 committed by Jose Lorenzo Rodriguez
parent 7a1275a1b1
commit 484c3ec688
2 changed files with 9 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
lib/Cake/Routing/Router.php
View file

@ -964,7 +964,7 @@ class Router {
}
$addition = http_build_query($q, null, $join);
if ($out && $addition) {
if ($out && $addition && substr($out, strlen($join) * -1, strlen($join)) != $join) {
$out .= $join;
}

8
lib/Cake/Test/Case/Routing/RouterTest.php
View file

@ -2626,5 +2626,13 @@ class RouterTest extends CakeTestCase {
$result = Router::queryString('foo=bar', array('php' => 'nut', 'jose' => 'zap'), true);
$expected = '?foo=bar&php=nut&jose=zap';
$this->assertEquals($expected, $result);
$result = Router::queryString('foo=bar&', array('php' => 'nut', 'jose' => 'zap'), true);
$expected = '?foo=bar&php=nut&jose=zap';
$this->assertEquals($expected, $result);
$result = Router::queryString('foo=bar&', array('php' => 'nut', 'jose' => 'zap'));
$expected = '?foo=bar&php=nut&jose=zap';
$this->assertEquals($expected, $result);
}
}