LP#1697954 Notify correct page of patron holds

Instead of notifying the grid on all holds, just notify when we're inside
the current page of results.

Signed-off-by: Mike Rylander <mrylander@gmail.com>
Signed-off-by: Bill Erickson <berickxx@gmail.com>

diff --git a/Open-ILS/web/js/ui/default/staff/circ/patron/holds.js b/Open-ILS/web/js/ui/default/staff/circ/patron/holds.js
index e2c422b..1e51cde 100644 (file)
--- a/Open-ILS/web/js/ui/default/staff/circ/patron/holds.js
+++ b/Open-ILS/web/js/ui/default/staff/circ/patron/holds.js
@@ -83,6 +83,7 @@ function($scope,  $q,  $routeParams,  egCore,  egUser,  patronSvc,
         if ($scope.holds_display == 'alt')
             method = 'open-ils.circ.holds.canceled.id_list.retrieve.authoritative';
 
+        var current = 0;
         egCore.net.request(
             'open-ils.circ', method,
             egCore.auth.token(), $scope.patron_id
@@ -92,7 +93,14 @@ function($scope,  $q,  $routeParams,  egCore,  egUser,  patronSvc,
 
             patronSvc.hold_ids = hold_ids;
             fetchHolds(offset, count)
-            .then(deferred.resolve, null, deferred.notify);
+            .then(deferred.resolve, null, function (data) {
+                if (data) {
+                    if (current >= offset && current < count) {
+                        deferred.notify(data);
+                    }
+                    current++;
+                }
+            });
         });
 
         return deferred.promise;