From 0315684082f556be11370a0944dfea4c2f47640c Mon Sep 17 00:00:00 2001 From: Jason Stephenson Date: Mon, 1 Oct 2012 13:08:38 -0400 Subject: [PATCH] Kludge for LP 929172. When marking a copy claims returned, we manually update the patron's claims returned count, even though a database trigger apparently does this for us. The trigger doesn't update the patron's last_xact_id, but updating via the perl code does. Signed-off-by: Jason Stephenson --- Open-ILS/src/perlmods/lib/OpenILS/Application/Circ.pm | 8 ++++++++ 1 file changed, 8 insertions(+) diff --git a/Open-ILS/src/perlmods/lib/OpenILS/Application/Circ.pm b/Open-ILS/src/perlmods/lib/OpenILS/Application/Circ.pm index 5c2a754a9b..018824fcf2 100644 --- a/Open-ILS/src/perlmods/lib/OpenILS/Application/Circ.pm +++ b/Open-ILS/src/perlmods/lib/OpenILS/Application/Circ.pm @@ -474,6 +474,14 @@ sub set_circ_claims_returned { } $e->update_action_circulation($circ) or return $e->die_event; + # Kludge for LP bug 929172: We set the patron's + # claims_returned_count and update the patron, even though the + # trigger does this for us. This updates the patron's + # last_xact_id. + # (https://bugs.launchpad.net/evergreen/+bug/929172) + my $crc = $patron->claims_returned_count + 1; + $patron->claims_returned_count($crc); + $e->update_actor_user($patron) or return $e->die_event; # see if there is a configured post-claims-return copy status if(my $stat = $U->ou_ancestor_setting_value($circ->circ_lib, 'circ.claim_return.copy_status')) { -- 2.11.0