You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@sling.apache.org by cz...@apache.org on 2014/01/29 18:23:21 UTC
svn commit: r1562515 -
/sling/trunk/bundles/resourceaccesssecurity/src/main/java/org/apache/sling/resourceaccesssecurity/impl/ResourceAccessSecurityImpl.java
Author: cziegeler
Date: Wed Jan 29 17:23:20 2014
New Revision: 1562515
URL: http://svn.apache.org/r1562515
Log:
Fix getReadbleResource
Modified:
sling/trunk/bundles/resourceaccesssecurity/src/main/java/org/apache/sling/resourceaccesssecurity/impl/ResourceAccessSecurityImpl.java
Modified: sling/trunk/bundles/resourceaccesssecurity/src/main/java/org/apache/sling/resourceaccesssecurity/impl/ResourceAccessSecurityImpl.java
URL: http://svn.apache.org/viewvc/sling/trunk/bundles/resourceaccesssecurity/src/main/java/org/apache/sling/resourceaccesssecurity/impl/ResourceAccessSecurityImpl.java?rev=1562515&r1=1562514&r2=1562515&view=diff
==============================================================================
--- sling/trunk/bundles/resourceaccesssecurity/src/main/java/org/apache/sling/resourceaccesssecurity/impl/ResourceAccessSecurityImpl.java (original)
+++ sling/trunk/bundles/resourceaccesssecurity/src/main/java/org/apache/sling/resourceaccesssecurity/impl/ResourceAccessSecurityImpl.java Wed Jan 29 17:23:20 2014
@@ -141,11 +141,14 @@ public abstract class ResourceAccessSecu
returnValue = null;
} else if (finalGateResult == GateResult.DONTCARE) {
returnValue = (this.defaultAllow ? resource : null);
- }
- // wrap Resource if read access is not or partly (values) not granted
- else if (!canReadAllValues) {
- returnValue = new AccessGateResourceWrapper(resource,
+ } else if (finalGateResult == GateResult.GRANTED ) {
+ // wrap Resource if read access is not or partly (values) not granted
+ if (!canReadAllValues) {
+ returnValue = new AccessGateResourceWrapper(resource,
accessGatesForValues);
+ } else {
+ returnValue = resource;
+ }
}
}
return returnValue;