LP#1406367 Fine generator skips no-fines transactions (parallel)
authorBill Erickson <berickxx@gmail.com>
Mon, 29 Dec 2014 22:05:39 +0000 (17:05 -0500)
committerJason Stephenson <jstephenson@mvlc.org>
Mon, 5 Jan 2015 21:45:21 +0000 (16:45 -0500)
Avoid inspecting no-fines transactions when running the fine generator
in parallel mode.

Signed-off-by: Bill Erickson <berickxx@gmail.com>
Signed-off-by: Jason Stephenson <jstephenson@mvlc.org>
Open-ILS/src/perlmods/lib/OpenILS/Application/Storage/Publisher/action.pm

index 56ad4a8..b619711 100644 (file)
@@ -311,7 +311,8 @@ sub grab_overdue {
 
     my $idlist = $self->api_name =~/id_list/o ? 1 : 0;
     
-    $client->respond( $idlist ? $_ : $_->to_fieldmapper ) for ( overdue_circs('', $idlist) );
+    $client->respond( $idlist ? $_ : $_->to_fieldmapper ) 
+        for ( overdue_circs('', $idlist, undef, 1) );
 
     return undef;
 
@@ -324,7 +325,8 @@ __PACKAGE__->register_method(
     signature       => q/
         Return list of overdue circulations and reservations to be used for fine generation.
         Despite the name, this is not a generic method for retrieving all overdue loans,
-        as it excludes loans that have already hit the maximum fine limit.
+        as it excludes loans that have already hit the maximum fine limit
+        and transactions which do not accrue fines.
 /,
 );
 __PACKAGE__->register_method(