Description: ------------ Please use http_build_query instead of string concatenation. It handles arrays and objects correctly and it's behavior is more likely to be expected. Reproduce code: --------------- //example $arr = array( 'apple', 'orange' ); $pager = Pager::factory( array( 'extraVars' => array( 'arr' => $arr ) ) ); $links = $pager->getLinks(); echo $links[ first ]; //old code foreach ($qs as $name => $value) { $querystring[] = $name . '=' . $value; } $querystring = array_merge($querystring, array_unique($arrays)); $querystring = array_map('htmlentities', $querystring); return '?' . implode('&', $querystring) . (!empty($querystring) ? '&' : '') . $this->_urlVar .'='; //new code return '?' . htmlentities( http_build_query( $qs ) ); Expected result: ---------------- http://host.tld/script.php?arr[]=apple&arr[]=orange&pageId=1 Actual result: -------------- http://host.tld/script.php?arr=Array&pageId=1

Code for loading http_build_query if it does not exist (so PHP_Compat is a dep only if php version < 5) if (!function_exists('http_build_query')) { require_once 'PHP/Compat.php'; PHP_Compat::loadFunction('http_build_query'); }

It makes sense. In the meanwhile I fixed the bug.

This bug has been fixed in CVS. In case this was a documentation problem, the fix will show up at the end of next Sunday (CET) on pear.php.net. In case this was a pear.php.net website problem, the change will show up on the website in short time. Thank you for the report, and for helping us make PEAR better. -- I ended up using a http_build_query() replacement since the standard function was too aggressive