From 662549b17d77c9863e1a41e324cc4ddb101a4722 Mon Sep 17 00:00:00 2001 From: Jonathan Druart Date: Mon, 22 Nov 2021 14:24:40 +0100 Subject: [PATCH] Bug 29524: (bug 28935 follow-up) Restore modification of some patron's attributes On commit 5f37d8d2f496ce3c9fd6dfd5a2efa7a9fe435af3 Bug 28935: No filtering on patron's data on member entry pages we restricted the list of the columns from the borrowers table that can be modified from the patron edit view. We were too restrictive, the following 3 attributes can be edited from this form: privacy_guarantor_fines, privacy_guarantor_checkouts, checkprevcheckout and lang Test plan: Turn on the following prefs: - AllowStaffToSetFinesVisibilityForGuarantor - AllowStaffToSetCheckoutsVisibilityForGuarantor - CheckPrevCheckout (set to 'unless overridden *') - TranslateNotices Edit a patron and see the 4 different options are now displayed. Change their value, save, edit again Confirm that the values have been saved --- members/memberentry.pl | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/members/memberentry.pl b/members/memberentry.pl index 50c86a4c137..38f2a2b1491 100755 --- a/members/memberentry.pl +++ b/members/memberentry.pl @@ -216,7 +216,7 @@ if ( $op eq 'insert' || $op eq 'modify' || $op eq 'save' || $op eq 'duplicate' ) # remove keys from %newdata that is not part of patron's attributes { my @keys_to_delete = ( - qr/^(borrowernumber|date_renewed|debarred|debarredcomment|flags|privacy|privacy_guarantor_fines|privacy_guarantor_checkouts|checkprevcheckout|updated_on|lastseen|lang|login_attempts|overdrive_auth_token|anonymized)$/, # Bug 28935 + qr/^(borrowernumber|date_renewed|debarred|debarredcomment|flags|privacy|updated_on|lastseen|login_attempts|overdrive_auth_token|anonymized)$/, # Bug 28935 qr/^BorrowerMandatoryField$/, qr/^category_type$/, qr/^check_member$/, -- 2.25.1