From b923c88f558adfa579b6dda6b8bd0c0b9ca98c4a Mon Sep 17 00:00:00 2001 From: Galen Charlton Date: Wed, 19 Jul 2017 11:12:29 -0400 Subject: [PATCH 1/1] LP#1669907: add comment about underlying cause of the bug Signed-off-by: Galen Charlton --- Open-ILS/web/js/ui/default/staff/cat/catalog/app.js | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/Open-ILS/web/js/ui/default/staff/cat/catalog/app.js b/Open-ILS/web/js/ui/default/staff/cat/catalog/app.js index 17eb562da1..e71f74f6b0 100644 --- a/Open-ILS/web/js/ui/default/staff/cat/catalog/app.js +++ b/Open-ILS/web/js/ui/default/staff/cat/catalog/app.js @@ -1514,6 +1514,10 @@ function($scope , $routeParams , $location , $window , $q , egCore , egHolds , e ).then( function(hold_data) { hold_ids = []; // clear the list of ids, hack to avoid dups + // TODO: fix the underlying problem, which is that + // this gets called twice when switching to the holds + // tab; once explicitly, and once via the change handler + // on the OU selector angular.forEach(hold_data, function(list, type) { hold_ids = hold_ids.concat(list); }); -- 2.43.2