This function is pretty cool and works well in all except one situation of mine.
/**
*
* @param string $url
* @param array $params
* @return string
*/
public static function getUrlWithUpdatedParams($url, $params) {
$uri = \League\Uri\Http::createFromString($url);
$query = \League\Uri\Components\Query::createFromParams($params);
return \League\Uri\merge_query($uri, $query)->__toString();
}
I'm using "league/uri": "^5.3"
(docs).
Here are my tests, where all assertions pass except the last one.
public function testGetUrlWithUpdatedParams() {
$this->assertEquals(url('/yo') . '?a=7', ST::getUrlWithUpdatedParams(url('/yo'), ['a' => 7]));
$this->assertEquals(url('/demo') . '?a=6&email=a@b.com', ST::getUrlWithUpdatedParams(url('/demo'), ['a' => 6, 'email' => 'a@b.com']));
$this->assertEquals(url('/sample') . '?ctrl=1&a=8', ST::getUrlWithUpdatedParams(url('/sample?ctrl=1'), ['a' => 8]));
$this->assertEquals(url('/sample') . '?ctrl=0&c=5', ST::getUrlWithUpdatedParams(url('/sample?ctrl=1'), ['c' => 5, 'ctrl' => 0]));
$this->assertEquals(url('/sample') . '?ctrl=1', ST::getUrlWithUpdatedParams(url('/sample?ctrl=1'), []));
$this->assertEquals(url('/sample') . '?ctrl=1', ST::getUrlWithUpdatedParams(url('/sample?ctrl=1'), ['ctrl' => null]));
$this->assertEquals(url('/sample') . '?ctrl=', ST::getUrlWithUpdatedParams(url('/sample?ctrl=1'), ['ctrl' => '']));
$encodedUrlAsParam = rawurlencode(url('/test'));
$this->assertEquals(url('/demo') . '?email_=a@b.com&url=' . $encodedUrlAsParam, ST::getUrlWithUpdatedParams(url('/demo'), ['email_' => 'a@b.com', 'url' => $encodedUrlAsParam])); //This one FAILS! Why in the world does getUrlWithUpdatedParams rawurlencode a second time??
}
I figured out that I could get the final assertion to pass by double rawurlencoding the param as follows:
$this->assertEquals(url('/demo') . '?email_=a@b.com&url=' . rawurlencode($encodedUrlAsParam), ST::getUrlWithUpdatedParams(url('/demo'), ['email_' => 'a@b.com', 'url' => $encodedUrlAsParam]));
But it feels weird to me to then be double decoding later when I want to retrieve the URL param from the main URL.
What's happening, and how can I avoid this?