[open-ils-commits] [GIT] Evergreen ILS branch rel_2_11 updated. 1d0e9ebff9887e8a10448e6e0a4f60311dd4860a

[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_11 has been updated
       via  1d0e9ebff9887e8a10448e6e0a4f60311dd4860a (commit)
      from  a4468573f5c3a878e61cd1ba05479697ad7b4696 (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 1d0e9ebff9887e8a10448e6e0a4f60311dd4860a
Author: Dan Wells <dbw2 at calvin.edu>
Date:   Tue Aug 30 21:50:19 2016 -0400

    LP#1618624 Fix thinko in adjust-to-zero
    
    It appears that this test was written for an 'unless' instead of
    an 'if'.  'if' seems just as natural here, so let's go that route.
    
    This should prevent us from closing circulations prematurely, and
    also close them when we need to.
    
    Signed-off-by: Dan Wells <dbw2 at calvin.edu>
    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 666b402..4b27e22 100644
--- a/Open-ILS/src/perlmods/lib/OpenILS/Application/Circ/Money.pm
+++ b/Open-ILS/src/perlmods/lib/OpenILS/Application/Circ/Money.pm
@@ -1051,7 +1051,7 @@ sub adjust_bills_to_zero_manual {
         # now we see if we can close the transaction
         # same logic as make_payments();
         my $circ = $e->retrieve_action_circulation($xact_id);
-        if ($circ and !$CC->can_close_circ($e, $circ)) {
+        if (!$circ or $CC->can_close_circ($e, $circ)) {
             # we don't check to see if the xact is already closed.  since the
             # xact had a negative balance, it should not have been closed, so
             # assume 'now' is the correct close time regardless.

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

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


hooks/post-receive
-- 
Evergreen ILS