#396 fix - Fixed more SonarQube issues

Roberto Sánchez

2014-09-19 8d5386be38db25a2a41c3bf6c876adee21ca26cc

 securis/src/main/java/net/curisit/securis/security/BasicSecurityContext.java
..
..
@@ -11,82 +11,83 @@
11
11
 
12
12
 public class BasicSecurityContext implements SecurityContext {
13
13
 
14
-	final public static String ROL_ADVANCE = "advance";
15
-	final public static String ROL_ADMIN = "admin";
14
+    final public static String ROL_ADVANCE = "advance";
15
+    final public static String ROL_ADMIN = "admin";
16
16
 
17
-	final static Map<String, Integer> ROLES = Utils.<String, Integer> createMap(ROL_ADVANCE, User.Rol.ADVANCE, ROL_ADMIN, User.Rol.ADMIN);
17
+    final static Map<String, Integer> ROLES = Utils.<String, Integer> createMap(ROL_ADVANCE, User.Rol.ADVANCE, ROL_ADMIN, User.Rol.ADMIN);
18
18
 
19
-	Principal user = null;
20
-	int roles = 0;
21
-	boolean secure = false;
22
-	Set<Integer> organizationsIds = null;
23
-	double ran = 0;
19
+    Principal user = null;
20
+    int roles = 0;
21
+    boolean secure = false;
22
+    Set<Integer> organizationsIds = null;
23
+    double ran = 0;
24
24
 
25
-	public BasicSecurityContext(String username, int roles, boolean secure) {
26
-		user = new UserPrincipal(username);
27
-		this.roles = roles;
28
-		this.secure = secure;
29
-		ran = Math.random();
30
-	}
25
+    public BasicSecurityContext(String username, int roles, boolean secure) {
26
+        user = new UserPrincipal(username);
27
+        this.roles = roles;
28
+        this.secure = secure;
29
+        ran = Math.random();
30
+    }
31
31
 
32
-	@Override
33
-	public Principal getUserPrincipal() {
34
-		return user;
35
-	}
32
+    @Override
33
+    public Principal getUserPrincipal() {
34
+        return user;
35
+    }
36
36
 
37
-	@Override
38
-	public boolean isUserInRole(String role) {
39
-		Integer introle = ROLES.get(role);
40
-		return introle != null && (introle & roles) != 0;
41
-	}
37
+    @Override
38
+    public boolean isUserInRole(String role) {
39
+        Integer introle = ROLES.get(role);
40
+        return introle != null && (introle & roles) != 0;
41
+    }
42
42
 
43
-	@Override
44
-	public boolean isSecure() {
45
-		return secure;
46
-	}
43
+    @Override
44
+    public boolean isSecure() {
45
+        return secure;
46
+    }
47
47
 
48
-	@Override
49
-	public String getAuthenticationScheme() {
50
-		return null;
51
-	}
48
+    @Override
49
+    public String getAuthenticationScheme() {
50
+        return null;
51
+    }
52
52
 
53
-	@Override
54
-	public String toString() {
53
+    @Override
54
+    public String toString() {
55
55
 
56
-		return String.format("SecurityContextWrapper(%f) %s", ran, user);
57
-	}
56
+        return String.format("SecurityContextWrapper(%f) %s", ran, user);
57
+    }
58
58
 
59
-	public void setOrganizationsIds(Set<Integer> orgs) {
60
-		this.organizationsIds = orgs;
61
-	}
59
+    public void setOrganizationsIds(Set<Integer> orgs) {
60
+        this.organizationsIds = orgs;
61
+    }
62
62
 
63
-	public Set<Integer> getOrganizationsIds() {
64
-		return this.organizationsIds;
65
-	}
63
+    public Set<Integer> getOrganizationsIds() {
64
+        return this.organizationsIds;
65
+    }
66
66
 
67
-	private class UserPrincipal implements Principal {
67
+    private class UserPrincipal implements Principal {
68
68
 
69
-		final String name;
69
+        final String name;
70
70
 
71
-		public UserPrincipal(String name) {
72
-			this.name = name;
73
-		}
71
+        public UserPrincipal(String name) {
72
+            this.name = name;
73
+        }
74
74
 
75
-		@Override
76
-		public String getName() {
77
-			return this.name;
78
-		}
75
+        @Override
76
+        public String getName() {
77
+            return this.name;
78
+        }
79
79
 
80
-		@Override
81
-		public String toString() {
82
-			return String.format("[%s]", name);
83
-		}
80
+        @Override
81
+        public String toString() {
82
+            return String.format("[%s]", name);
83
+        }
84
84
 
85
-	}
85
+    }
86
86
 
87
-	public boolean isOrgAccesible(Integer orgid) {
88
-		if (organizationsIds == null || orgid == null)
89
-			return false;
90
-		return organizationsIds.contains(orgid);
91
-	}
87
+    public boolean isOrgAccesible(Integer orgid) {
88
+        if (organizationsIds == null || orgid == null) {
89
+            return false;
90
+        }
91
+        return organizationsIds.contains(orgid);
92
+    }
92
93
 }