LP#1526163-auto-renew-not-honored user/bhorn/LP1526163-checkout-auto-renew-not-honored
authorBilly Horn <bhorn@catalystdevworks.com>
Fri, 18 Nov 2016 21:04:24 +0000 (13:04 -0800)
committerBilly Horn <bhorn@catalystdevworks.com>
Thu, 5 Jan 2017 17:00:44 +0000 (09:00 -0800)
add check for auto_renew policy
add check to checkout to see if it is a
  renew when updating the items out count

Signed-off-by: Billy Horn <bhorn@catalystdevworks.com>
Open-ILS/web/js/ui/default/staff/circ/patron/checkout.js
Open-ILS/web/js/ui/default/staff/circ/services/circ.js

index 3f82de7..8fe8610 100644 (file)
@@ -148,8 +148,11 @@ function($scope , $q , $routeParams , egCore , egUser , patronSvc ,
             function(co_resp) {
                 // update stats locally so we don't have to fetch them w/
                 // each checkout.
-                patronSvc.patron_stats.checkouts.out++;
-                patronSvc.patron_stats.checkouts.total_out++;
+                //check for renew so that is doesn't update incorrectly
+                if(co_resp.evt[0].payload.parent_circ == null){
+                    patronSvc.patron_stats.checkouts.out++;
+                    patronSvc.patron_stats.checkouts.total_out++;
+                }
 
                 // copy the response event into the original grid row item
                 // note: angular.copy clobbers the destination
index 753f54b..64145d4 100644 (file)
@@ -439,6 +439,9 @@ function($uibModal , $q , egCore , egAlertDialog , egConfirmDialog,
                 return service.precat_dialog(params, options);
 
             case 'OPEN_CIRCULATION_EXISTS':
+                if(evt[0].payload.auto_renew == 1){
+                    return service.renew(params, options);
+                }
                 return service.circ_exists_dialog(evt, params, options);
 
             case 'COPY_IN_TRANSIT':