From 692097038ac5ebce6e12e485c9b5710b832a333d Mon Sep 17 00:00:00 2001
From: Mike Rylander <mrylander@gmail.com>
Date: Tue, 17 Feb 2015 17:14:23 -0500
Subject: [PATCH] Test value directly, and invert test for the a-disabled test

Signed-off-by: Mike Rylander <mrylander@gmail.com>
---
 .../src/templates/staff/circ/share/t_hold_edit_pickup_lib.tt2     | 2 +-
 Open-ILS/web/js/ui/default/staff/circ/services/holds.js           | 2 +-
 Open-ILS/web/js/ui/default/staff/services/org.js                  | 8 ++++----
 3 files changed, 6 insertions(+), 6 deletions(-)

diff --git a/Open-ILS/src/templates/staff/circ/share/t_hold_edit_pickup_lib.tt2 b/Open-ILS/src/templates/staff/circ/share/t_hold_edit_pickup_lib.tt2
index b70b9ce16e..513bfa1e37 100644
--- a/Open-ILS/src/templates/staff/circ/share/t_hold_edit_pickup_lib.tt2
+++ b/Open-ILS/src/templates/staff/circ/share/t_hold_edit_pickup_lib.tt2
@@ -10,7 +10,7 @@
     <div class="form-group">
       <div class="col-md-4">[% l('Select Library:') %]</div>
       <div class="col-md-8">
-        <eg-org-selector disable-test="can_be_pickup" selected="args.org_unit"></eg-org-selector>
+        <eg-org-selector disable-test="cant_be_pickup" selected="args.org_unit"></eg-org-selector>
       </div>
     </div>
   </div>
diff --git a/Open-ILS/web/js/ui/default/staff/circ/services/holds.js b/Open-ILS/web/js/ui/default/staff/circ/services/holds.js
index af255b2541..e3e620647e 100644
--- a/Open-ILS/web/js/ui/default/staff/circ/services/holds.js
+++ b/Open-ILS/web/js/ui/default/staff/circ/services/holds.js
@@ -198,7 +198,7 @@ function($modal , $q , egCore , egConfirmDialog , egAlertDialog) {
             controller : 
                 ['$scope', '$modalInstance',
                 function($scope, $modalInstance) {
-                    $scope.can_be_pickup = egCore.org.CanHaveUsers;
+                    $scope.cant_be_pickup = function (id) { return !egCore.org.CanHaveUsers(id); };
                     $scope.args = {};
                     $scope.ok = function() { 
                         var vals = hold_ids.map(function(hold_id) {
diff --git a/Open-ILS/web/js/ui/default/staff/services/org.js b/Open-ILS/web/js/ui/default/staff/services/org.js
index dd89d825b5..dcf1d52948 100644
--- a/Open-ILS/web/js/ui/default/staff/services/org.js
+++ b/Open-ILS/web/js/ui/default/staff/services/org.js
@@ -43,18 +43,18 @@ function($q,  egEnv,  egAuth,  egNet) {
 
     // tests that a node can have users
     service.CanHaveUsers = function(node_or_id) {
-        return Boolean(service
+	return service
             .get(node_or_id)
             .ou_type()
-            .can_have_users());
+            .can_have_users() == 't';
     }
 
     // tests that a node can have volumes
     service.CanHaveVolumes = function(node_or_id) {
-        return Boolean(service
+        return service
             .get(node_or_id)
             .ou_type()
-            .can_have_vols());
+            .can_have_vols() == 't';
     }
 
     // list of org_unit objects  or IDs for me + descendants
-- 
2.11.0