[Koha-bugs] [Bug 15809] versions of CGI < 4.08 do not have multi_param

[bugzilla-daemon at bugs.koha-community.org]

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15809

--- Comment #13 from Marcel de Rooy <m.de.rooy at rijksmuseum.nl> ---
(In reply to Jonathan Druart from comment #12)
> (In reply to Galen Charlton from comment #10)
> 
> I totally agree with the "what to do next" part, but what about now? My
> patch seems to match your requirements.

Jonathan, I am reading: "I am not keen about monkey patching a core module",
which imo does not seem to translate to "match your requirements" :)
I would not opt for a complete Koha::CGI at this point. But I would (slightly)
prefer the Koha::CGI::multi_param above "*CGI::multi_param = \&CGI::param". It
will make any developer immediately aware of our "interception". The switchover
to the real multi_param at some point in time will be completely painless.

Also note the different handling of the LIST_CONTEXT_WARN var.

> It could be very easy and quick to replace the different occurrences. I
> would like to avoid a temporary solution. Even if we need one, something
> different than "FIXME bug 2505" would be better ;)
Agreed, altough some code is temporary in any solution for now.

-- 
You are receiving this mail because:
You are watching all bug changes.