[jira] [Commented] (SLING-2803) Create a Validation module capable of validating POST requests or Resources

["Bertrand Delacretaz (JIRA)" <ji...@apache.org>]

    [ 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