From 986cc8e8910966e61b3dd09da170d9ebe63c478c Mon Sep 17 00:00:00 2001 From: Nick Clemens Date: Wed, 26 Feb 2020 03:16:26 +0000 Subject: [PATCH] Bug 24733: created_by param should be an arrayref We allow for the user to add multiple users to this field, and the search function expects and array. We have been treating it as a scalar TO test: 1 - Have an open basket in acquisitions 2 - Add to basket - from an existing order (copy) 3 - Enter a name in the 'Basket created by' field and select a user 4 - Hit search 5 - ISE (Can't use string ("1") as an ARRAY ref while "strict refs" in use at /usr/share/koha/lib/C4/Acquisition.pm line 2482.) 6 - Apply patch 7 - Repeat 1-4 8 - Success! 9 - Test with mutiple 'created by' entries as well Signed-off-by: Jonathan Druart Signed-off-by: Martin Renvoize (cherry picked from commit 789866e585be0a5c439559383325cbe4db525412) Signed-off-by: Lucas Gass (cherry picked from commit 63374a8db6f759c8bb3e9b6ca1029ad07d2bb938) Signed-off-by: Hayley Mapley --- acqui/duplicate_orders.pl | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/acqui/duplicate_orders.pl b/acqui/duplicate_orders.pl index 549c8b495a..c3b86b82ec 100755 --- a/acqui/duplicate_orders.pl +++ b/acqui/duplicate_orders.pl @@ -65,8 +65,9 @@ my $filters = { orderstatus => scalar $input->param('orderstatus'), ordernumber => scalar $input->param('ordernumber'), search_children_too => scalar $input->param('search_children_too'), - created_by => scalar $input->multi_param('created_by'), + created_by => [$input->multi_param('created_by')] }; + my $from_placed_on = eval { dt_from_string( scalar $input->param('from') ) } || dt_from_string; my $to_placed_on = -- 2.39.5