From b746411600bdd210a5ee4abb1ffa194b6c4f2a0d Mon Sep 17 00:00:00 2001 From: Bill Erickson Date: Thu, 29 Jun 2017 17:53:33 -0400 Subject: [PATCH 1/1] LP#1694529 Claims never checked out updates patron last-xact-id Force an upate to the user last_xact_id value when marking an item claims never checked out, since the process results in patron record changes. Signed-off-by: Bill Erickson Signed-off-by: Galen Charlton --- .../perlmods/lib/OpenILS/Application/Circ/Circulate.pm | 10 ++++++++++ 1 file changed, 10 insertions(+) diff --git a/Open-ILS/src/perlmods/lib/OpenILS/Application/Circ/Circulate.pm b/Open-ILS/src/perlmods/lib/OpenILS/Application/Circ/Circulate.pm index 21b875d49c..ce05bc0c2f 100644 --- a/Open-ILS/src/perlmods/lib/OpenILS/Application/Circ/Circulate.pm +++ b/Open-ILS/src/perlmods/lib/OpenILS/Application/Circ/Circulate.pm @@ -290,6 +290,16 @@ sub run_method { } else { + # checkin and reservation return can result in modifications to + # actor.usr.claims_never_checked_out_count without also modifying + # actor.last_xact_id. Perform a no-op update on the patron to + # force an update to last_xact_id. + if ($circulator->claims_never_checked_out && $circulator->patron) { + $circulator->editor->update_actor_user( + $circulator->editor->retrieve_actor_user($circulator->patron->id)) + or return $circulator->editor->die_event; + } + $circulator->editor->commit; } -- 2.43.2