From 8d5386be38db25a2a41c3bf6c876adee21ca26cc Mon Sep 17 00:00:00 2001
From: Roberto Sánchez <roberto.sanchez@curisit.net>
Date: Fri, 19 Sep 2014 08:26:02 +0000
Subject: [PATCH] #396 fix - Fixed more SonarQube issues
---
securis/src/main/java/net/curisit/securis/security/BasicSecurityContext.java | 121 ++++++++++++++++++++--------------------
1 files changed, 61 insertions(+), 60 deletions(-)
diff --git a/securis/src/main/java/net/curisit/securis/security/BasicSecurityContext.java b/securis/src/main/java/net/curisit/securis/security/BasicSecurityContext.java
index 73ef3b2..3f20ddc 100644
--- a/securis/src/main/java/net/curisit/securis/security/BasicSecurityContext.java
+++ b/securis/src/main/java/net/curisit/securis/security/BasicSecurityContext.java
@@ -11,82 +11,83 @@
public class BasicSecurityContext implements SecurityContext {
- final public static String ROL_ADVANCE = "advance";
- final public static String ROL_ADMIN = "admin";
+ final public static String ROL_ADVANCE = "advance";
+ final public static String ROL_ADMIN = "admin";
- final static Map<String, Integer> ROLES = Utils.<String, Integer> createMap(ROL_ADVANCE, User.Rol.ADVANCE, ROL_ADMIN, User.Rol.ADMIN);
+ final static Map<String, Integer> ROLES = Utils.<String, Integer> createMap(ROL_ADVANCE, User.Rol.ADVANCE, ROL_ADMIN, User.Rol.ADMIN);
- Principal user = null;
- int roles = 0;
- boolean secure = false;
- Set<Integer> organizationsIds = null;
- double ran = 0;
+ Principal user = null;
+ int roles = 0;
+ boolean secure = false;
+ Set<Integer> organizationsIds = null;
+ double ran = 0;
- public BasicSecurityContext(String username, int roles, boolean secure) {
- user = new UserPrincipal(username);
- this.roles = roles;
- this.secure = secure;
- ran = Math.random();
- }
+ public BasicSecurityContext(String username, int roles, boolean secure) {
+ user = new UserPrincipal(username);
+ this.roles = roles;
+ this.secure = secure;
+ ran = Math.random();
+ }
- @Override
- public Principal getUserPrincipal() {
- return user;
- }
+ @Override
+ public Principal getUserPrincipal() {
+ return user;
+ }
- @Override
- public boolean isUserInRole(String role) {
- Integer introle = ROLES.get(role);
- return introle != null && (introle & roles) != 0;
- }
+ @Override
+ public boolean isUserInRole(String role) {
+ Integer introle = ROLES.get(role);
+ return introle != null && (introle & roles) != 0;
+ }
- @Override
- public boolean isSecure() {
- return secure;
- }
+ @Override
+ public boolean isSecure() {
+ return secure;
+ }
- @Override
- public String getAuthenticationScheme() {
- return null;
- }
+ @Override
+ public String getAuthenticationScheme() {
+ return null;
+ }
- @Override
- public String toString() {
+ @Override
+ public String toString() {
- return String.format("SecurityContextWrapper(%f) %s", ran, user);
- }
+ return String.format("SecurityContextWrapper(%f) %s", ran, user);
+ }
- public void setOrganizationsIds(Set<Integer> orgs) {
- this.organizationsIds = orgs;
- }
+ public void setOrganizationsIds(Set<Integer> orgs) {
+ this.organizationsIds = orgs;
+ }
- public Set<Integer> getOrganizationsIds() {
- return this.organizationsIds;
- }
+ public Set<Integer> getOrganizationsIds() {
+ return this.organizationsIds;
+ }
- private class UserPrincipal implements Principal {
+ private class UserPrincipal implements Principal {
- final String name;
+ final String name;
- public UserPrincipal(String name) {
- this.name = name;
- }
+ public UserPrincipal(String name) {
+ this.name = name;
+ }
- @Override
- public String getName() {
- return this.name;
- }
+ @Override
+ public String getName() {
+ return this.name;
+ }
- @Override
- public String toString() {
- return String.format("[%s]", name);
- }
+ @Override
+ public String toString() {
+ return String.format("[%s]", name);
+ }
- }
+ }
- public boolean isOrgAccesible(Integer orgid) {
- if (organizationsIds == null || orgid == null)
- return false;
- return organizationsIds.contains(orgid);
- }
+ public boolean isOrgAccesible(Integer orgid) {
+ if (organizationsIds == null || orgid == null) {
+ return false;
+ }
+ return organizationsIds.contains(orgid);
+ }
}
--
Gitblit v1.3.2