You are viewing a plain text version of this content. The canonical link for it is here.

Posted to oak-commits@jackrabbit.apache.org by st...@apache.org on 2017/07/21 08:59:26 UTC

svn commit: r1802554 - /jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/security/user/UserPrincipalProvider.java

Author: stillalex
Date: Fri Jul 21 08:59:26 2017
New Revision: 1802554

URL: http://svn.apache.org/viewvc?rev=1802554&view=rev
Log:
OAK-6470 Remove extra call to createGroupPrincipal in UserPrincipalProvider


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

Modified: jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/security/user/UserPrincipalProvider.java
URL: http://svn.apache.org/viewvc/jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/security/user/UserPrincipalProvider.java?rev=1802554&r1=1802553&r2=1802554&view=diff
==============================================================================
--- jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/security/user/UserPrincipalProvider.java (original)
+++ jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/security/user/UserPrincipalProvider.java Fri Jul 21 08:59:26 2017
@@ -254,7 +254,7 @@ class UserPrincipalProvider implements P
                 if (groupTree != null && UserUtil.isType(groupTree, AuthorizableType.GROUP)) {
                     Group gr = createGroupPrincipal(groupTree);
                     if (gr != null) {
-                        groupPrincipals.add(createGroupPrincipal(groupTree));
+                        groupPrincipals.add(gr);
                     }
                 }
             }