BZ-1175682: a bad fix for BZ-1122594, blocked any user that had 'user'

or 'manager' roles
kiegroup · Jan 14, 2015 · 85ce297 · 85ce297
1 parent cc2a268
commit 85ce297
Showing 1 changed file with 4 additions and 5 deletions.
diff --git a/...org/kie/workbench/common/services/backend/security/KieFileSystemAuthorizationManager.java b/...org/kie/workbench/common/services/backend/security/KieFileSystemAuthorizationManager.java
@@ -23,11 +23,10 @@ public boolean authorize( final Resource resource,
     }
 
     private boolean checkRole( final User subject ) {
-        if ( subject.getRoles().contains( new RoleImpl( USER.toString() ) ) || subject.getRoles().contains( new RoleImpl( MANAGER.toString() ) ) ) {
-            if ( subject.getRoles().contains( new RoleImpl( ADMIN.toString() ) ) || subject.getRoles().contains( new RoleImpl( DEVELOPER.toString() ) ) ||
-                    subject.getRoles().contains( new RoleImpl( ANALYST.toString() ) ) ) {
-                return true;
-            }
+        if ( subject.getRoles().contains( new RoleImpl( ADMIN.toString() ) ) ||
+                subject.getRoles().contains( new RoleImpl( DEVELOPER.toString() ) ) ||
+                subject.getRoles().contains( new RoleImpl( ANALYST.toString() ) ) ) {
+            return true;
         }
         return false;
     }