Author: kwin
Date: Tue Feb 28 07:53:30 2017
New Revision: 1784698
URL: http://svn.apache.org/viewvc?rev=1784698&view=rev
Log:
use generics for Predicate, fix some warnings
Modified:
sling/trunk/bundles/extensions/validation/api/src/main/java/org/apache/sling/validation/ValidationService.java
sling/trunk/bundles/extensions/validation/core/src/main/java/org/apache/sling/validation/impl/ValidationModelRetrieverImpl.java
sling/trunk/bundles/extensions/validation/core/src/main/java/org/apache/sling/validation/impl/ValidationResourceVisitor.java
sling/trunk/bundles/extensions/validation/core/src/main/java/org/apache/sling/validation/impl/model/ChildResourceBuilder.java
Modified: sling/trunk/bundles/extensions/validation/api/src/main/java/org/apache/sling/validation/ValidationService.java
URL: http://svn.apache.org/viewvc/sling/trunk/bundles/extensions/validation/api/src/main/java/org/apache/sling/validation/ValidationService.java?rev=1784698&r1=1784697&r2=1784698&view=diff
==============================================================================
--- sling/trunk/bundles/extensions/validation/api/src/main/java/org/apache/sling/validation/ValidationService.java
(original)
+++ sling/trunk/bundles/extensions/validation/api/src/main/java/org/apache/sling/validation/ValidationService.java
Tue Feb 28 07:53:30 2017
@@ -96,6 +96,6 @@ public interface ValidationService {
* @throws IllegalArgumentException in case resourceType is absolute but outside of the
search paths or if no validation model could be found (and enforceValidation is {@code true}).
* @throws SlingValidationException if one validator was called with invalid arguments
*/
- @Nonnull ValidationResult validateResourceRecursively(@Nonnull Resource resource, boolean
enforceValidation, Predicate filter, boolean considerResourceSuperTypeModels) throws IllegalStateException,
IllegalArgumentException, SlingValidationException;
+ @Nonnull ValidationResult validateResourceRecursively(@Nonnull Resource resource, boolean
enforceValidation, Predicate<Resource> filter, boolean considerResourceSuperTypeModels)
throws IllegalStateException, IllegalArgumentException, SlingValidationException;
}
Modified: sling/trunk/bundles/extensions/validation/core/src/main/java/org/apache/sling/validation/impl/ValidationModelRetrieverImpl.java
URL: http://svn.apache.org/viewvc/sling/trunk/bundles/extensions/validation/core/src/main/java/org/apache/sling/validation/impl/ValidationModelRetrieverImpl.java?rev=1784698&r1=1784697&r2=1784698&view=diff
==============================================================================
--- sling/trunk/bundles/extensions/validation/core/src/main/java/org/apache/sling/validation/impl/ValidationModelRetrieverImpl.java
(original)
+++ sling/trunk/bundles/extensions/validation/core/src/main/java/org/apache/sling/validation/impl/ValidationModelRetrieverImpl.java
Tue Feb 28 07:53:30 2017
@@ -30,7 +30,6 @@ import org.apache.sling.api.resource.Log
import org.apache.sling.api.resource.ResourceResolver;
import org.apache.sling.api.resource.ResourceResolverFactory;
import org.apache.sling.commons.osgi.RankedServices;
-import org.apache.sling.serviceusermapping.ServiceUserMapped;
import org.apache.sling.validation.impl.model.MergedValidationModel;
import org.apache.sling.validation.impl.util.Trie;
import org.apache.sling.validation.model.ValidationModel;
Modified: sling/trunk/bundles/extensions/validation/core/src/main/java/org/apache/sling/validation/impl/ValidationResourceVisitor.java
URL: http://svn.apache.org/viewvc/sling/trunk/bundles/extensions/validation/core/src/main/java/org/apache/sling/validation/impl/ValidationResourceVisitor.java?rev=1784698&r1=1784697&r2=1784698&view=diff
==============================================================================
--- sling/trunk/bundles/extensions/validation/core/src/main/java/org/apache/sling/validation/impl/ValidationResourceVisitor.java
(original)
+++ sling/trunk/bundles/extensions/validation/core/src/main/java/org/apache/sling/validation/impl/ValidationResourceVisitor.java
Tue Feb 28 07:53:30 2017
@@ -35,9 +35,9 @@ public class ValidationResourceVisitor e
private final boolean enforceValidation;
private final boolean considerResourceSuperTypeModels;
private final @Nonnull CompositeValidationResult result;
- private final Predicate filter;
+ private final Predicate<Resource> filter;
- public ValidationResourceVisitor(ValidationServiceImpl validationService, String rootResourcePath,
boolean enforceValidation, Predicate filter, boolean considerResourceSuperTypeModels) {
+ public ValidationResourceVisitor(ValidationServiceImpl validationService, String rootResourcePath,
boolean enforceValidation, Predicate<Resource> filter, boolean considerResourceSuperTypeModels)
{
super();
this.validationService = validationService;
this.rootResourcePath = rootResourcePath + "/";
@@ -60,7 +60,7 @@ public class ValidationResourceVisitor e
}
// calculate the property name correctly from the root
// the relative path must not end with a slash and not start with a slash
- final String relativePath;
+ @Nonnull final String relativePath;
if (resource.getPath().startsWith(rootResourcePath)) {
relativePath = resource.getPath().substring(rootResourcePath.length());
} else {
Modified: sling/trunk/bundles/extensions/validation/core/src/main/java/org/apache/sling/validation/impl/model/ChildResourceBuilder.java
URL: http://svn.apache.org/viewvc/sling/trunk/bundles/extensions/validation/core/src/main/java/org/apache/sling/validation/impl/model/ChildResourceBuilder.java?rev=1784698&r1=1784697&r2=1784698&view=diff
==============================================================================
--- sling/trunk/bundles/extensions/validation/core/src/main/java/org/apache/sling/validation/impl/model/ChildResourceBuilder.java
(original)
+++ sling/trunk/bundles/extensions/validation/core/src/main/java/org/apache/sling/validation/impl/model/ChildResourceBuilder.java
Tue Feb 28 07:53:30 2017
@@ -31,7 +31,9 @@ public class ChildResourceBuilder {
public boolean optional;
public boolean multiple;
String nameRegex;
+ @Nonnull
private final List<ResourceProperty> resourceProperties;
+ @Nonnull
private final List<ChildResource> children;
public ChildResourceBuilder() {
|