[dev] patch to Horde::addParameter()
Michael M Slusarz
slusarz@bigworm.colorado.edu
Fri, 5 Jul 2002 18:20:49 -0600
Quoting David Ulevitch <davidu@everydns.net>:
| Here's a small patch to let Horde::addParameter support alternate output
| delim's.
|
| The change is a switch from: & to ini_get('arg_separator.output').
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 been trying to clean this up (using Horde::addParameter()) everywhere
I see it, but since I mainly deal with IMP and Horde there are many other
places that haven't been touched yet.
michael
______________________________________________
Michael Slusarz [slusarz@bigworm.colorado.edu]
The University of Colorado at Boulder