You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@jackrabbit.apache.org by an...@apache.org on 2010/12/07 16:09:24 UTC

svn commit: r1043084 - /jackrabbit/trunk/jackrabbit-core/src/main/java/org/apache/jackrabbit/core/security/user/UserImpl.java

Author: angela
Date: Tue Dec  7 15:09:24 2010
New Revision: 1043084

URL: http://svn.apache.org/viewvc?rev=1043084&view=rev
Log:
minor improvement (order of methods)

Modified:
    jackrabbit/trunk/jackrabbit-core/src/main/java/org/apache/jackrabbit/core/security/user/UserImpl.java

Modified: jackrabbit/trunk/jackrabbit-core/src/main/java/org/apache/jackrabbit/core/security/user/UserImpl.java
URL: http://svn.apache.org/viewvc/jackrabbit/trunk/jackrabbit-core/src/main/java/org/apache/jackrabbit/core/security/user/UserImpl.java?rev=1043084&r1=1043083&r2=1043084&view=diff
==============================================================================
--- jackrabbit/trunk/jackrabbit-core/src/main/java/org/apache/jackrabbit/core/security/user/UserImpl.java (original)
+++ jackrabbit/trunk/jackrabbit-core/src/main/java/org/apache/jackrabbit/core/security/user/UserImpl.java Tue Dec  7 15:09:24 2010
@@ -61,7 +61,29 @@ public class UserImpl extends Authorizab
         }
     }
 
-    //------------------------------------------------< User >------------------
+    //-------------------------------------------------------< Authorizable >---
+    /**
+     * @see org.apache.jackrabbit.api.security.user.Authorizable#isGroup()
+     */
+    public boolean isGroup() {
+        return false;
+    }
+
+    /**
+     * @see org.apache.jackrabbit.api.security.user.Authorizable#getPrincipal()
+     */
+    public Principal getPrincipal() throws RepositoryException {
+        if (principal == null) {
+            if (isAdmin()) {
+                principal = new NodeBasedAdminPrincipal(getPrincipalName());
+            } else {
+                principal = new NodeBasedPrincipal(getPrincipalName());
+            }
+        }
+        return principal;
+    }
+
+    //---------------------------------------------------------------< User >---
     /**
      * @see User#isAdmin()
      */
@@ -88,26 +110,6 @@ public class UserImpl extends Authorizab
             throw new RepositoryException(e);
         }
     }
-    /**
-     * @see User#isGroup()
-     */
-    public boolean isGroup() {
-        return false;
-    }
-
-    /**
-     * @see User#getPrincipal()
-     */
-    public Principal getPrincipal() throws RepositoryException {
-        if (principal == null) {
-            if (isAdmin()) {
-                principal = new NodeBasedAdminPrincipal(getPrincipalName());
-            } else {
-                principal = new NodeBasedPrincipal(getPrincipalName());
-            }
-        }
-        return principal;
-    }
 
     /**
      * @see User#getImpersonation()