[ 
https://issues.apache.org/jira/browse/SLING-2803?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13698982#comment-13698982
 ] 

Bertrand Delacretaz commented on SLING-2803:
--------------------------------------------

I briefly reviewed the validation-2.zip code, here's some comments - I also 
have a few conceptual things to discuss, will do that on the dev list.

ValidationModel.getValidatedResourceType - shouldn't a model apply to 1..N 
resource types?

Does ValidationResult apply to a single Field? If not it should IMO return a 
list of failures, not just a single message.

ValidationResult.getValidationResultFailureMessage can be called just 
getFailureMessage()

public enum Type: can't you use standard java types for that?

Shouldn't ValidatorLookupService provide an Iterator over existing Validators?
                
> Create a Validation module capable of validating POST requests or Resources
> ---------------------------------------------------------------------------
>
>                 Key: SLING-2803
>                 URL: https://issues.apache.org/jira/browse/SLING-2803
>             Project: Sling
>          Issue Type: New Feature
>            Reporter: Radu Cotescu
>         Attachments: org.apache.sling.validation-2.zip, 
> org.apache.sling.validation.zip
>
>
> Several Sling users have asked in the past [1] about a Validation mechanism 
> in Sling that's able to validate data coming from the user. However, such a 
> module is still missing.
> This issue is meant to track the work progress on such a module.
> Ideally the module should be able to validate both request parameters and 
> Resources, by obtaining their properties via a ValueMap and provide feedback 
> in case of a validation error.
> [1] - http://sling.markmail.org/search/?q=validation

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

Reply via email to