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 17eb562..e71f74f 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
@@ -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);
                 });