From 1c9135262b9233f59fd05e34c90980e7dc8be338 Mon Sep 17 00:00:00 2001 From: Fridolyn SOMERS Date: Mon, 17 Jun 2013 17:01:33 +0200 Subject: [PATCH] Bug 10483: fix item uniqueness check on order form When syspref "UniqueItemFields" is defined, the item uniqueness is tested in acquisition by an AJAX call to check_uniqueness.pl. This patch fixes an issue where check_uniqueness.pl wasn't looking at the correct CGI parameters. Test plan : - Select an existing item with barcode - Add "barcode" to "UniqueItemFields" syspref (use space as separator for multiple values) - Set "AcqCreateItem" syspref to "Create an item when placing an order" - Go to an acquisition basket - Create a new order from empty - Enter existing barcode in item form and save => You get an alert that barcode already exists and order is not saved - Enter a non-existing barcode in item form and save => Order and item are created Signed-off-by: Kyle M Hall Signed-off-by: Chris Cormack Signed-off-by: Galen Charlton --- acqui/check_uniqueness.pl | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/acqui/check_uniqueness.pl b/acqui/check_uniqueness.pl index 626004868a..8deb1959c6 100755 --- a/acqui/check_uniqueness.pl +++ b/acqui/check_uniqueness.pl @@ -37,8 +37,8 @@ use C4::Output; use C4::Items; my $input = new CGI; -my @field = $input->param('field'); -my @value = $input->param('value'); +my @field = $input->param('field[]'); +my @value = $input->param('value[]'); my $r = {}; my $i = 0; -- 2.39.5