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/17 18:16:33 UTC

svn commit: r1483899 - /jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/security/authorization/permission/PermissionProviderImpl.java

Author: angela
Date: Fri May 17 16:16:33 2013
New Revision: 1483899

URL: http://svn.apache.org/r1483899
Log:
OAK-527: permissions (add TODO wrt readstatus for access control items)

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

Modified: jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/security/authorization/permission/PermissionProviderImpl.java
URL: http://svn.apache.org/viewvc/jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/security/authorization/permission/PermissionProviderImpl.java?rev=1483899&r1=1483898&r2=1483899&view=diff
==============================================================================
--- jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/security/authorization/permission/PermissionProviderImpl.java (original)
+++ jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/security/authorization/permission/PermissionProviderImpl.java Fri May 17 16:16:33 2013
@@ -115,6 +115,7 @@ public class PermissionProviderImpl impl
                 return ReadStatus.ALLOW_ALL;
             case TreeTypeProvider.TYPE_AC:
                 // TODO: review if read-ac permission is never fine-granular
+                // TODO: replace by calling #getReadStatus
                 return canReadAccessControlContent(tree, null) ? ReadStatus.ALLOW_ALL : ReadStatus.DENY_ALL;
             case TreeTypeProvider.TYPE_VERSION:
                 return getVersionContentReadStatus(tree, property);