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) {