From de41a7de104f846e9d2b3bc967d0cd9de0173341 Mon Sep 17 00:00:00 2001 From: Jonathan Druart Date: Mon, 24 Jun 2019 09:58:00 -0500 Subject: [PATCH] Bug 23082: Fatal error editing a restricted patron In members/memberentry.pl we have to explicitely remove the template's params that are not patron's attributes. Certainly caused by commit 1bb6cec902088d07c7a43addc4b4adb5b0bf3d4f Bug 20287: Fix update of patrons, clean the data before ->store Test plan: Create a restriction for a patron Edit patron's details Remove the restriction => Without this patch you get "No property remove_debarment for Koha::Patron" => With this patch applied the restriction is removed Signed-off-by: Mark Tompsett Signed-off-by: Kyle M Hall Signed-off-by: Martin Renvoize (cherry picked from commit 8947d5939fb33efd0887a6f443d392b9df17e2aa) Signed-off-by: Fridolin Somers (cherry picked from commit 0d2ebac7649f35c9273c5181e81a21f654e71f85) Signed-off-by: Lucas Gass --- members/memberentry.pl | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/members/memberentry.pl b/members/memberentry.pl index 61a1368607..93c92c8f37 100755 --- a/members/memberentry.pl +++ b/members/memberentry.pl @@ -221,7 +221,7 @@ if ( $op eq 'insert' || $op eq 'modify' || $op eq 'save' || $op eq 'duplicate' ) qr/^\d+-DAYS/, qr/^patron_attr_/, qr/^csrf_token$/, - qr/^add_debarment$/, qr/^debarred_expiration$/, # We already dealt with debarments previously + qr/^add_debarment$/, qr/^debarred_expiration$/, qr/^remove_debarment$/, # We already dealt with debarments previously qr/^housebound_chooser$/, qr/^housebound_deliverer$/, qr/^select_city$/, ); -- 2.39.5