From f35fa3d4541a83e8cafeaf5f396e4aacc684e63f Mon Sep 17 00:00:00 2001 From: Kyle M Hall Date: Fri, 22 Nov 2013 08:31:31 -0500 Subject: [PATCH] Bug 11282: fix overwriting of existing restrictions when adding new one This patch fixes a bug where using the patron editor to add a new restriction overwrote the first existing one. Test Plan: 1) Edit a patron, add a restriction 2) Edit the patron again, add a second restriction 3) Note the first restriction has disappeared! 4) Apply this patch 5) Edit the patron again, add another restriction 6) Note the previous restriction is not longer removed Signed-off-by: Jonathan Druart Signed-off-by: Katrin Fischer Tested: - Adding and removing multiple restrictions from - the details tab - the checkouts tab - the edit patron form All works as expected. Patch passes all tests in t, xt, and QA script. Signed-off-by: Galen Charlton --- members/memberentry.pl | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/members/memberentry.pl b/members/memberentry.pl index 0d3a3c3792..591aa7c56f 100755 --- a/members/memberentry.pl +++ b/members/memberentry.pl @@ -110,7 +110,7 @@ if ( $input->param('add_debarment') ) { { 'dt' => dt_from_string($expiration), 'dateformat' => 'iso' } ) : undef; - AddUniqueDebarment( + AddDebarment( { borrowernumber => $borrowernumber, type => 'MANUAL', -- 2.39.5