You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@sling.apache.org by fm...@apache.org on 2014/01/30 09:06:50 UTC
svn commit: r1562713 -
/sling/trunk/bundles/resourceresolver/src/main/java/org/apache/sling/resourceresolver/impl/helper/ResourceResolverContext.java
Author: fmeschbe
Date: Thu Jan 30 08:06:50 2014
New Revision: 1562713
URL: http://svn.apache.org/r1562713
Log:
SLING-3350 Adapt to simplified Features service API
(no need for the ClientContext any longer)
Modified:
sling/trunk/bundles/resourceresolver/src/main/java/org/apache/sling/resourceresolver/impl/helper/ResourceResolverContext.java
Modified: sling/trunk/bundles/resourceresolver/src/main/java/org/apache/sling/resourceresolver/impl/helper/ResourceResolverContext.java
URL: http://svn.apache.org/viewvc/sling/trunk/bundles/resourceresolver/src/main/java/org/apache/sling/resourceresolver/impl/helper/ResourceResolverContext.java?rev=1562713&r1=1562712&r2=1562713&view=diff
==============================================================================
--- sling/trunk/bundles/resourceresolver/src/main/java/org/apache/sling/resourceresolver/impl/helper/ResourceResolverContext.java (original)
+++ sling/trunk/bundles/resourceresolver/src/main/java/org/apache/sling/resourceresolver/impl/helper/ResourceResolverContext.java Thu Jan 30 08:06:50 2014
@@ -33,7 +33,6 @@ import org.apache.sling.api.resource.Res
import org.apache.sling.api.resource.ResourceResolverFactory;
import org.apache.sling.api.resource.ResourceUtil;
import org.apache.sling.api.resource.ValueMap;
-import org.apache.sling.featureflags.ClientContext;
import org.apache.sling.featureflags.Features;
import org.apache.sling.resourceresolver.impl.ResourceAccessSecurityTracker;
@@ -283,20 +282,20 @@ public class ResourceResolverContext {
if (resource != null) {
Object featuresService = this.featuresHolder.getFeatures();
if (featuresService instanceof Features) {
- String[] features = getProperty(resource, RESOURCE_PROPERTY, String[].class);
- if (features != null && features.length > 0) {
- ClientContext featureContext = ((Features) featuresService).getCurrentClientContext();
- for (String feature : features) {
+ Features features = (Features) featuresService;
+ String[] featureNames = getProperty(resource, RESOURCE_PROPERTY, String[].class);
+ if (featureNames != null && featureNames.length > 0) {
+ for (String featureName : featureNames) {
// check whether the feature must be disabled
boolean negative = false;
- if (feature.charAt(0) == '-') {
- feature = feature.substring(1);
+ if (featureName.charAt(0) == '-') {
+ featureName = featureName.substring(1);
negative = true;
}
- if (featureContext.isEnabled(feature) ^ negative) {
- resource.getResourceMetadata().put(RESOURCE_PROPERTY, features);
+ if (features.isEnabled(featureName) ^ negative) {
+ resource.getResourceMetadata().put(RESOURCE_PROPERTY, featureNames);
return resource;
}
}