From: Jason Stephenson Date: Sat, 10 Dec 2016 02:21:35 +0000 (-0500) Subject: LP 1648922: Hide orgs that can't have users in workstation registration. X-Git-Url: https://old-git.evergreen-ils.org/?a=commitdiff_plain;h=608b5dea1ea9936d49d6405a9949b8b854779d70;p=working%2FEvergreen.git LP 1648922: Hide orgs that can't have users in workstation registration. A user can register a workstation at an org unit that can't have users in the webstaff client. The XUL staff client did not allow this. Such org units were grayed out in the selector. This commit hides org units that can't have users in the ou selector for workstation registration in the webstaff client. This is in addition to hiding the existing hiding of org units where the user does not have the REGISTER_WORKSTATION permission. To reproduce this bug, see that org units that have false for can_have_users appear in the selection for workstation registration. Also see that you can register a workstation at one. After applying this patch, make sure you've logged out and then log in to the webstaff client. You will now see that those org units no longer appear in the slection list. You cannot register workstations at those org units any longer. Signed-off-by: Jason Stephenson --- diff --git a/Open-ILS/web/js/ui/default/staff/admin/workstation/app.js b/Open-ILS/web/js/ui/default/staff/admin/workstation/app.js index dd51bc8129..b59c9a0644 100644 --- a/Open-ILS/web/js/ui/default/staff/admin/workstation/app.js +++ b/Open-ILS/web/js/ui/default/staff/admin/workstation/app.js @@ -665,9 +665,10 @@ function($scope , $q , $window , $location , egCore , egAlertDialog , workstatio reg_perm_orgs = orgList; // hide orgs in the context org selector where this login - // does not have the reg_ws perm + // does not have the reg_ws perm or the org can't have users $scope.wsOrgHidden = function(id) { - return reg_perm_orgs.indexOf(id) == -1; + return reg_perm_orgs.indexOf(id) == -1 + || $scope.cant_have_users(id); } // fetch the locally stored workstation data