LP#1669907: add comment about underlying cause of the bug
authorGalen Charlton <gmc@equinoxinitiative.org>
Wed, 19 Jul 2017 15:12:29 +0000 (11:12 -0400)
committerGalen Charlton <gmc@equinoxinitiative.org>
Wed, 19 Jul 2017 15:13:21 +0000 (11:13 -0400)
Signed-off-by: Galen Charlton <gmc@equinoxinitiative.org>
Open-ILS/web/js/ui/default/staff/cat/catalog/app.js

index 27eb25f..c8b4950 100644 (file)
@@ -1513,6 +1513,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);
                 });