Hi all:
Been banging my head with this one. Some of my code broke after the
last update to then current CVS. Ended up being changes to the url
view helper. If you pass in parameters as null, they should be removed
by the assemble method (undocumented feature but I assume it's
intended). however, recent changes to urlencode the urlOptions broke
this behaviour. the php function urlencode returns an empty string. So
the route->assemble method doesn't know anymore that it was null. I
suggest the following fix:
if ($encode) {
foreach ($urlOptions as $key => $option) {
if(null === $option){
$urlOptions[$key] = $option;
}else{
$urlOptions[$key] = urlencode($option);
}
}
}
Unless there is an issue with unencoded null values (which I can't
picture), this should solve the issue without causing any
compatibility issues.
Anybody out there that can commit this fix?
Thanks,
Gunter
