From 20e11d747dd0a75700a1ae4c17da8264cecb0bc0 Mon Sep 17 00:00:00 2001 From: Erik Amaru Ortiz Date: Thu, 8 Mar 2012 10:45:58 -0400 Subject: [PATCH] =?UTF-8?q?BUG=208664=20"Con=20la=20opci=C3=B3n=20user=20e?= =?UTF-8?q?xperience=20se=20puede=20aplicar=20al=20us..=20Adm..."=20SOLVED?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit - missing validation - validation added for users that have admins role --- workflow/engine/templates/admin/uxUsersList.js | 9 ++++----- 1 file changed, 4 insertions(+), 5 deletions(-) diff --git a/workflow/engine/templates/admin/uxUsersList.js b/workflow/engine/templates/admin/uxUsersList.js index 8b5e6ae4c..d8859352d 100755 --- a/workflow/engine/templates/admin/uxUsersList.js +++ b/workflow/engine/templates/admin/uxUsersList.js @@ -175,12 +175,11 @@ Ext.onReady(function(){ select: function(a, b) { var row = usersGrid.getSelectionModel().getSelected(); role = row.get('USR_ROLE'); - //console.log(role) - // if (role == 'PROCESSMAKER_ADMIN') { - // PMExt.warning(_('ID_ERROR'), 'You can\'t assign this User Experience UI for users that have PROCESSMAKER_ADMIN role.'); - // this.setValue('NORMAL'); - // } + if (role == 'PROCESSMAKER_ADMIN' && (this.value == 'SIMPLIFIED' || this.value == 'SINGLE')) { + PMExt.warning(_('ID_WARNING'), _('ID_ADMINS_CANT_USE_UXS')); + this.setValue('NORMAL'); + } } } })