Greetings,

My only concern is aging code. In the mean time, I think it is totally functional as a nice quick fix.

Additionally, as we patch these params with multi_params, perhaps we should better document paths of execution to evaluate whether it really is a multi_param or just a param. And if we need that single value in an array for some reason, why not:

my @foobars;
my $foobar = $query->param('foobar') // q{};
push @foobars, $foobar;

This is clear that the foobar param is scalar, and only pushes a clean value into the foobars array.

GPML,
Mark Tompsett

_______________________________________________
Koha-devel mailing list
[email protected]
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-devel
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

Reply via email to