Tickets :: Comment on [#4602] patch for maintenance

* Your Email Address
* Spam protection	Enter the letters below: . ..__..__ . ..__ \|\ \|\| \|[__)\|\/\|\| \ \| \\|\|__\|\| \| \|\|__/
Comment	> This doesn't look right. What is happening I think is this - the > redirect URL is something like: > > foo.php?foo1=bar1&app=imp > > > > Then we add another parameter via addParameter() so the redirect URL > looks like this: > > foo.php?foo1=bar1&app=imp&foo2=bar2 > > > > So we are mixing and matching & and & because addParameter() > doesn't recognize we are already using & in the query string. > And we can't encode all &'s as &amp's, as your patch does, since this > won't work with the Redirect calls in some browsers. > > > > So what I have done is rewritten addParameter() to auto detect the > current delimiter and deal with it in a consistent manner. So > feedback on this change is needed to see if that fixes this issue.
Attachment
Watch this ticket