https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15809 --- Comment #14 from Jonathan Druart <jonathan.druart@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.