From: Galen Charlton <gmc@equinoxinitiative.org>
Date: Wed, 19 Jul 2017 15:12:29 +0000 (-0400)
Subject: LP#1669907: add comment about underlying cause of the bug
X-Git-Url: https://old-git.evergreen-ils.org/?a=commitdiff_plain;h=b923c88f558adfa579b6dda6b8bd0c0b9ca98c4a;p=evergreen%2Fmasslnc.git

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 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);
                 });