[open-ils-commits] ***SPAM*** [GIT] Evergreen ILS branch rel_2_7 updated. 7998ca2858ed709fc78639fa641d91b6ca271d07

[Evergreen Git]

This is an automated email from the git hooks/post-receive script. It was
generated because a ref change was pushed to the repository containing
the project "Evergreen ILS".

The branch, rel_2_7 has been updated
       via  7998ca2858ed709fc78639fa641d91b6ca271d07 (commit)
      from  0afd7a6594308cfe17f1e6e7ceb720440a110c13 (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.

- Log -----------------------------------------------------------------
commit 7998ca2858ed709fc78639fa641d91b6ca271d07
Author: Ben Shum <bshum at biblio.org>
Date:   Wed Jan 21 16:30:53 2015 -0500

    LP#1412893: Only apply lost and paid status with the proper transactions
    
    As discovered while testing the lost and paid feature, when closing earlier transactions where
    the item is currently lost, it could change the item status to lost and paid prematurely.
    
    Change the logic so that it only does this if the current transaction being closed either has
    a status of lost or longoverdue.
    
    Signed-off-by: Ben Shum <bshum at biblio.org>
    Signed-off-by: Kathy Lussier <klussier at masslnc.org>

diff --git a/Open-ILS/src/perlmods/lib/OpenILS/Application/Circ/Money.pm b/Open-ILS/src/perlmods/lib/OpenILS/Application/Circ/Money.pm
index 0401876..6c59ad1 100644
--- a/Open-ILS/src/perlmods/lib/OpenILS/Application/Circ/Money.pm
+++ b/Open-ILS/src/perlmods/lib/OpenILS/Application/Circ/Money.pm
@@ -497,7 +497,7 @@ sub make_payments {
                 # check org_unit_settings for the copy owning library
                 # and adjust and possibly adjust copy status to lost
                 # and paid.
-                if ($circ) {
+                if ($circ && ($circ->stop_fines eq 'LOST' || $circ->stop_fines eq 'LONGOVERDUE')) {
                     # We need the copy to check settings and to possibly
                     # change its status.
                     my $copy = $circ->target_copy();

-----------------------------------------------------------------------

Summary of changes:
 .../perlmods/lib/OpenILS/Application/Circ/Money.pm |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)


hooks/post-receive
-- 
Evergreen ILS