[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