LP#1669907: add comment about underlying cause of the bug

Signed-off-by: Galen Charlton <gmc@equinoxinitiative.org>

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 27eb25f..c8b4950 100644 (file)
--- a/Open-ILS/web/js/ui/default/staff/cat/catalog/app.js
+++ b/Open-ILS/web/js/ui/default/staff/cat/catalog/app.js
@@ -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);
                 });