From d65748ec24447da1af2d97ad524d97d602c34fb6 Mon Sep 17 00:00:00 2001 From: Hammat Wele Date: Thu, 14 Dec 2023 13:59:42 +0000 Subject: [PATCH] Bug 35508: Update borrowers.updated_on when modifying a patron's attribute Currently, if you have patron attributes and modify the values in a patron's account, the patron's updated_on date is not updated. This patch makes the "Updated on" change when a patron attribute is updated. To test: 1. Create a patron attribute type 1.1. Go to Administration > Patron attribute types 1.2. Click New patron attribute 1.3. Fill out the code and description 1.4. Click Save 2. Edit a patron (normal) 2.1. Go to Patrons and find a patron account 2.2. Click Edit 2.3. Change a regular field (e.g. Middle name) 2.4. Click Save --> Notice the "Updated on" date in the left column has been updated to now 3. Edit a patron attribute 3.1. In another patron account*, click Edit 3.2. Change the value of an attribute 3.3. Click Save --> Notice the "Updated on" date did not change 4. Apply the patch 4.1 Repeat step 3.1, 3.2, 3.3 --> Notice the "Updated on" date has now changed --- members/memberentry.pl | 1 + 1 file changed, 1 insertion(+) diff --git a/members/memberentry.pl b/members/memberentry.pl index 57e780cfec..df3662aff6 100755 --- a/members/memberentry.pl +++ b/members/memberentry.pl @@ -582,6 +582,7 @@ if ((!$nok) and $nodouble and ($op eq 'insert' or $op eq 'save')){ if (C4::Context->preference('ExtendedPatronAttributes') and $input->param('setting_extended_patron_attributes')) { $patron->extended_attributes->filter_by_branch_limitations->delete; $patron->extended_attributes($extended_patron_attributes); + $patron->set( { updated_on => "" } )->store; } if ( $destination eq 'circ' and not C4::Auth::haspermission( C4::Context->userenv->{id}, { circulate => 'circulate_remaining_permissions' } ) ) { -- 2.34.1