From 6f99b8992699bc006d22a513171ff22995506baf Mon Sep 17 00:00:00 2001 From: Michael Jumper Date: Fri, 16 Oct 2015 22:47:49 -0700 Subject: [PATCH] GUAC-1193: Fix logic error caused by missing braces. Add braces for readability elsewhere. --- .../webapp/app/navigation/services/userPageService.js | 9 ++++++--- 1 file changed, 6 insertions(+), 3 deletions(-) diff --git a/guacamole/src/main/webapp/app/navigation/services/userPageService.js b/guacamole/src/main/webapp/app/navigation/services/userPageService.js index 9001aa1e0..933f89fb6 100644 --- a/guacamole/src/main/webapp/app/navigation/services/userPageService.js +++ b/guacamole/src/main/webapp/app/navigation/services/userPageService.js @@ -211,8 +211,9 @@ angular.module('navigation').factory('userPageService', ['$injector', // Permission to administer users || PermissionSet.hasUserPermission(permissions, PermissionSet.ObjectPermissionType.ADMINISTER) - ) + ) { canManageUsers.push(dataSource); + } // Determine whether the current user needs access to the connection management UI if ( @@ -232,16 +233,18 @@ angular.module('navigation').factory('userPageService', ['$injector', // Permission to administer connections or connection groups || PermissionSet.hasConnectionPermission(permissions, PermissionSet.ObjectPermissionType.ADMINISTER) || PermissionSet.hasConnectionGroupPermission(permissions, PermissionSet.ObjectPermissionType.ADMINISTER) - ) + ) { canManageConnections.push(dataSource); + } // Determine whether the current user needs access to the session management UI or view connection history if ( // A user must be a system administrator to manage sessions PermissionSet.hasSystemPermission(permissions, PermissionSet.SystemPermissionType.ADMINISTER) - ) + ) { canManageSessions.push(dataSource); canViewConnectionRecords.push(dataSource); + } });