updated coding rules.

[utils] / security / src / main / java / org / wamblee / usermgt / JaasUserAccessor.java

diff --git a/security/src/main/java/org/wamblee/usermgt/JaasUserAccessor.java b/security/src/main/java/org/wamblee/usermgt/JaasUserAccessor.java
index f353e3b121bf63cc6905fff8bb197358b51809fe..8102b82fe22a6dd447b920d2abbf6f41869e3575 100644 (file)
--- a/security/src/main/java/org/wamblee/usermgt/JaasUserAccessor.java
+++ b/security/src/main/java/org/wamblee/usermgt/JaasUserAccessor.java
@@ -33,12 +33,12 @@ public class JaasUserAccessor implements UserAccessor {
     /**
      * User administration to use. 
      */
-    private UserAdministration _admin;
+    private UserAdministration admin;
 
     /**
      * Class of the JAAS user principal. 
      */
-    private Class _userPrincipalClass;
+    private Class userPrincipalClass;
 
     /**
      * Constructs user accessor. 
@@ -46,10 +46,10 @@ public class JaasUserAccessor implements UserAccessor {
      * @param aUserClassName Class name of the user principal. 
      */
     public JaasUserAccessor(UserAdministration aAdmin, String aUserClassName) {
-        _admin = aAdmin;
+        admin = aAdmin;
         try {
-            _userPrincipalClass = Class.forName(aUserClassName);
-            if ( !Principal.class.isAssignableFrom(_userPrincipalClass)) {
+            userPrincipalClass = Class.forName(aUserClassName);
+            if ( !Principal.class.isAssignableFrom(userPrincipalClass)) {
                 throw new IllegalArgumentException("Specified class '" + aUserClassName + "' is not a subclass of '" +
                         Principal.class.getName());
             }
@@ -70,7 +70,7 @@ public class JaasUserAccessor implements UserAccessor {
         }
         Principal userPrincipal = getUserPrincipal(subject);
       
-        return _admin.getUser(userPrincipal.getName());
+        return admin.getUser(userPrincipal.getName());
     }
 
     /**
@@ -83,17 +83,17 @@ public class JaasUserAccessor implements UserAccessor {
         Set<Principal> principals = subject.getPrincipals();
         Principal userPrincipal = null;  
         for ( Principal principal: principals) { 
-            if ( principal.getClass().equals(_userPrincipalClass)) { 
+            if ( principal.getClass().equals(userPrincipalClass)) { 
                 if ( userPrincipal != null ) { 
                     throw new IllegalArgumentException(
-                            "Multiple principals for class '" + _userPrincipalClass + "', subject: " + subject);                   
+                            "Multiple principals for class '" + userPrincipalClass + "', subject: " + subject);                   
                 }
                 userPrincipal = principal; 
             }
         }
         if ( userPrincipal == null ) { 
             throw new IllegalArgumentException(
-                    "No user principal found for class '" + _userPrincipalClass + "', subject: " + subject);
+                    "No user principal found for class '" + userPrincipalClass + "', subject: " + subject);
         }
         return userPrincipal;
     }