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 an...@apache.org on 2013/05/28 12:32:02 UTC

svn commit: r1486858 - /jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/security/authorization/AccessControlManagerImpl.java

Author: angela
Date: Tue May 28 10:32:01 2013
New Revision: 1486858

URL: http://svn.apache.org/r1486858
Log:
OAK-51 : Access Control Management (wip)

Modified:
    jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/security/authorization/AccessControlManagerImpl.java

Modified: jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/security/authorization/AccessControlManagerImpl.java
URL: http://svn.apache.org/viewvc/jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/security/authorization/AccessControlManagerImpl.java?rev=1486858&r1=1486857&r2=1486858&view=diff
==============================================================================
--- jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/security/authorization/AccessControlManagerImpl.java (original)
+++ jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/security/authorization/AccessControlManagerImpl.java Tue May 28 10:32:01 2013
@@ -572,7 +572,6 @@ public class AccessControlManagerImpl im
 
     @Nonnull
     private static Result searchAces(@Nonnull Set<Principal> principals, @Nonnull Root root) throws RepositoryException {
-        // TODO: specify sort order
         StringBuilder stmt = new StringBuilder("/jcr:root");
         stmt.append("//element(*,");
         stmt.append(NT_REP_ACE);
@@ -590,6 +589,7 @@ public class AccessControlManagerImpl im
             i++;
         }
         stmt.append(']');
+        stmt.append(" order by jcr:path");
 
         try {
             QueryEngine queryEngine = root.getQueryEngine();