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

--- Comment #14 from Jonathan Druart <[email protected]> 
---
(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.
_______________________________________________
Koha-bugs mailing list
[email protected]
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

Reply via email to