[dev] patch to Horde::addParameter()

Chuck Hagenbuch chuck@horde.org
Fri, 5 Jul 2002 21:16:19 -0400


Quoting Michael M Slusarz <slusarz@bigworm.colorado.edu>:

> I think this is a good idea.  Unfortunately, we don't (in a _whole_ lot of
> places) use addParameter to build URLs.  That could conceivably mean that
> a URL could have a combination of separator characters (e.g. '&' and ';') 
> which is not good.

I've committed the change; the places where we aren't using 
Horde::addParameter() won't work if the argument separator is set to ; 
anyway; this can just be more encouragement to get things cleaned up.

-chuck

--
Charles Hagenbuch, <chuck@horde.org>
"After a few minutes the most aromatic and nice smelling Italian coffee 
 will come out of the exhaustpipe." - Our stove-top espresso pot