You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@sling.apache.org by ol...@apache.org on 2016/05/14 12:41:32 UTC

svn commit: r1743821 - /sling/trunk/bundles/extensions/validation/core/src/main/java/org/apache/sling/validation/impl/CompositeValidationResult.java

Author: olli
Date: Sat May 14 12:41:32 2016
New Revision: 1743821

URL: http://svn.apache.org/viewvc?rev=1743821&view=rev
Log:
remove inline comment, add javadoc

Modified:
    sling/trunk/bundles/extensions/validation/core/src/main/java/org/apache/sling/validation/impl/CompositeValidationResult.java

Modified: sling/trunk/bundles/extensions/validation/core/src/main/java/org/apache/sling/validation/impl/CompositeValidationResult.java
URL: http://svn.apache.org/viewvc/sling/trunk/bundles/extensions/validation/core/src/main/java/org/apache/sling/validation/impl/CompositeValidationResult.java?rev=1743821&r1=1743820&r2=1743821&view=diff
==============================================================================
--- sling/trunk/bundles/extensions/validation/core/src/main/java/org/apache/sling/validation/impl/CompositeValidationResult.java (original)
+++ sling/trunk/bundles/extensions/validation/core/src/main/java/org/apache/sling/validation/impl/CompositeValidationResult.java Sat May 14 12:41:32 2016
@@ -47,9 +47,11 @@ public class CompositeValidationResult i
         results.add(new DefaultValidationResult(location, severity, message, messageArguments));
     }
 
+    /**
+     * @return false if at least one {@link ValidationResult} is not valid, true otherwise
+     */
     @Override
     public boolean isValid() {
-        // this is only valid if all aggregated results are valid
         for (ValidationResult result : results) {
             if (!result.isValid()) {
                 return false;