makary/phpbb
1
0
Fork 0
mirror of https://github.com/phpbb/phpbb.git synced 2025-06-28 22:28:51 +00:00
Code Issues Projects Releases Packages Wiki Activity

Merge pull request #2919 from Senky/ticket/12852

Browse source 
[ticket/12852] Make get_url_parts handle get variable with no value

* Senky/ticket/12852:
  [ticket/12852] Add unit tests
  [ticket/12852] Add space after if
  [ticket/12852] Remove whitespace
  [ticket/12852] Make get_url_parts handle get variable with no value
This commit is contained in:
Tristan Darricau 2014-09-06 15:47:30 +02:00
commit 297ab5c526
2 changed files with 48 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

20
phpBB/phpbb/path_helper.php
View file

@ -316,16 +316,24 @@ class path_helper
* Glue URL parameters together
*
* @param array $params URL parameters in the form of array(name => value)
* @return string Returns the glued string, e.g. name1=value1&name2=value2
* @return string Returns the glued string, e.g. name1=value1&name2&name3=value3
*/
public function glue_url_params($params)
{
$_params = array();
foreach ($params as $key => $value)
{
// some parameters do not have value
if ($value !== null)
{
$_params[] = $key . '=' . $value;
}
else
{
$_params[] = $key;
}
}
return implode('&', $_params);
}
@ -353,7 +361,17 @@ class path_helper
{
continue;
}
// some parameters don't have value
if (strpos($argument, '=') !== false)
{
list($key, $value) = explode('=', $argument, 2);
}
else
{
$key = $argument;
$value = null;
}
if ($key === '')
{

27
tests/path_helper/path_helper_test.php
View file

@ -205,6 +205,18 @@ class phpbb_path_helper_test extends phpbb_test_case
array('test' => 'xyz', 'var' => 'value'),
'test=xyz&var=value',
),
array(
array('test' => null),
'test',
),
array(
array('test' => null, 'var' => null),
'test&var',
),
array(
array('test' => 'xyz', 'var' => null, 'bar' => 'value'),
'test=xyz&var&bar=value',
),
);
}
@ -254,6 +266,21 @@ class phpbb_path_helper_test extends phpbb_test_case
true,
array('base' => 'mcp.php', 'params' => array('f' => '3')),
),
array(
'index.php?ready',
false,
array('base' => 'index.php', 'params' => array('ready' => null)),
),
array(
'index.php?i=1&ready',
true,
array('base' => 'index.php', 'params' => array('i' => '1', 'ready' => null)),
),
array(
'index.php?ready&i=1',
false,
array('base' => 'index.php', 'params' => array('ready' => null, 'i' => '1')),
),
);
}