[dev] rtrim/Horde::removeParameter PATCH

[Marko]

Quoting Chuck Hagenbuch <chuck at horde.org>:

> > it's messing up the Horde::removeParameter() - when the parameter removed
> 
> Then removeParameter() should be fixed to honor the current arg separator.
> Patch?
Have a patch!

Essentially this now honours any arg separator, it takes care of arg separators
being already in htmlentities format, and it takes care of not leaving any
trailing args or ?... unfortunately it still can't make my morning coffee...

it is a couple of lines longer, but it is the best solution i could find that
seemed to take care of _any_ eventuality as far as i could tell. things like
parse_str and simpler matching string didn't work a) because of htmlentities and
b) because of i didn't like the fact that '.' would get changed to '_' in param
names without any way to glue the query back together again in its original form.

someone give it a spin?

m.-------------- next part --------------
A non-text attachment was scrubbed...
Name: Horde.php.patch
Type: application/octet-stream
Size: 2217 bytes
Desc: not available
Url : http://lists.horde.org/archives/dev/attachments/20021229/ba196c43/Horde.php.obj