LP#1709521 Apply recent patrons from checkout
authorBill Erickson <berickxx@gmail.com>
Fri, 1 Sep 2017 01:14:32 +0000 (21:14 -0400)
committerKathy Lussier <klussier@masslnc.org>
Fri, 1 Sep 2017 12:25:11 +0000 (08:25 -0400)
Load the max recent patrons setting directly in the patron service
instead of requring the calling UI to apply it.  This fixes an issue
where recent patrons are not set from the checkout-by-barcode
interface.

Signed-off-by: Bill Erickson <berickxx@gmail.com>
Signed-off-by: Kathy Lussier <klussier@masslnc.org>
Open-ILS/web/js/ui/default/staff/services/patron_search.js

index 21e477d..c27274f 100644 (file)
@@ -48,6 +48,18 @@ function($q , $timeout , $location , egCore,  egUser , egConfirmDialog , $locale
     }
     service.resetPatronLists();  // initialize
 
+    // Max recents setting is loaded and scrubbed during egStartup.
+    // Copy it to a local variable here for ease of local access
+    // after startup has run.
+    egCore.startup.go().then(
+        function() {
+            egCore.org.settings('ui.staff.max_recent_patrons')
+            .then(function(s) {
+                service.maxRecentPatrons = s['ui.staff.max_recent_patrons'];
+            });
+        }
+    );
+
     // Returns true if the last alerted patron matches the current
     // patron.  Otherwise, the last alerted patron is set to the 
     // current patron and false is returned.
@@ -509,12 +521,6 @@ function($scope,  $q,  $routeParams,  $timeout,  $window,  $location,  egCore,
         selectedItems : function() {return []}
     }
 
-    // Max recents setting is loaded and scrubbed during egStartup.
-    // Copy it to a local variable here for ease of local access.
-    egCore.org.settings('ui.staff.max_recent_patrons').then(function(s) {
-        patronSvc.maxRecentPatrons = s['ui.staff.max_recent_patrons'];
-    });
-
     // The first time we encounter the show-recent CGI param, put the
     // service into show-recent mode.  The first time recents are shown,
     // the service is taken out of show-recent mode so the page does not