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/04/23 11:58:42 UTC

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

Author: angela
Date: Tue Apr 23 09:58:42 2013
New Revision: 1470869

URL: http://svn.apache.org/r1470869
Log:
OAK-64: privilege mgt (tmp fix for test failure caused by hidden items not being accessible)

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=1470869&r1=1470868&r2=1470869&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 Tue Apr 23 09:58:42 2013
@@ -110,7 +110,7 @@ public class PermissionProviderImpl impl
         switch (type) {
             case TreeTypeProvider.TYPE_HIDDEN:
                 // TODO: OAK-753 decide on where to filter out hidden items.
-                return ReadStatus.DENY_ALL;
+                return ReadStatus.ALLOW_ALL;
             case TreeTypeProvider.TYPE_AC:
                 // TODO: review if read-ac permission is never fine-granular
                 return canReadAccessControlContent(tree, null) ? ReadStatus.ALLOW_ALL : ReadStatus.DENY_ALL;
@@ -132,7 +132,7 @@ public class PermissionProviderImpl impl
         switch (type) {
             case TreeTypeProvider.TYPE_HIDDEN:
                 // TODO: OAK-753 decide on where to filter out hidden items.
-                return false;
+                return true;
             case TreeTypeProvider.TYPE_VERSION:
                 TreeLocation location = getVersionableLocation(tree, property);
                 if (location == null) {