From a0d989091da997d6c571c9ed611f3784df54107d Mon Sep 17 00:00:00 2001 From: Chris Sharp Date: Wed, 22 Apr 2015 09:54:11 -0400 Subject: [PATCH] LP#1446860 - Correct mistaken logic in previous fix. See https://bugs.launchpad.net/evergreen/+bug/1446860/comments/2 for reference. The logic was not enforcing the "if the editing staff member doesn't have permission to edit a user in this permission group" check. This corrects that. Signed-off-by: Chris Sharp Signed-off-by: Ben Shum --- Open-ILS/web/js/ui/default/actor/user/register.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Open-ILS/web/js/ui/default/actor/user/register.js b/Open-ILS/web/js/ui/default/actor/user/register.js index e5846fc948..2e6f2ad2d1 100644 --- a/Open-ILS/web/js/ui/default/actor/user/register.js +++ b/Open-ILS/web/js/ui/default/actor/user/register.js @@ -302,7 +302,7 @@ function load() { dojo.connect(setExpireDate, 'onClick', setExpireDateHandler); - if(!patron.isnew() && !checkGrpAppPerm(patron.profile()) && patron.id() == openils.User.user.id()) { + if(!patron.isnew() && !checkGrpAppPerm(patron.profile()) || patron.id() == openils.User.user.id()) { // we are not allowed to edit this user, so disable the save option saveButton.attr('disabled', true); saveCloneButton.attr('disabled', true); -- 2.43.2