[open-ils-commits] ***SPAM*** [GIT] Evergreen ILS branch master updated. 609db84f0499cf7083b09d1786ace1a60f71986e

[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, master has been updated
       via  609db84f0499cf7083b09d1786ace1a60f71986e (commit)
      from  eda58b7612de8c7438d545b6b97693b751d46e4b (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 609db84f0499cf7083b09d1786ace1a60f71986e
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