[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 #14 from Jonathan Druart <jonathan.druart at bugs.koha-community.org> ---
(In reply to Marcel de Rooy from comment #13)
> (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" :)

Sorry, I thought Galen was talking about patching CGI (solution suggested on
the ML).

> 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.

Is anyone ready to submit a counter-patch?

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