[ 
http://www.stripesframework.org/jira/browse/STS-752?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12066#action_12066
 ] 

Frederic Daoud commented on STS-752:
------------------------------------

> just b/c a value is supplied does not mean that the value is adequate from a 
> type conversion perspective

True, but if the value supplied is not adequate from a type conversion 
perspective, the type converter should be responsible for indicating the error. 
The type converter does get a reference to the collection of validation errors 
in the convert method.

> @Validate(required=true) only applies to parameters pre-conversion
> ------------------------------------------------------------------
>
>                 Key: STS-752
>                 URL: http://www.stripesframework.org/jira/browse/STS-752
>             Project: Stripes
>          Issue Type: Bug
>          Components: Validation
>            Reporter: Aaron Porter
>            Assignee: Aaron Porter
>            Priority: Minor
>             Fix For: Release 1.6
>
>
> Validation for required parameters passes when a parameter is not null and 
> not empty even if the type converter returns null. It seems that if a 
> property is marked with @Validate(required=true) then the type converter 
> returning null should trigger a validation error.

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: 
http://www.stripesframework.org/jira/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

------------------------------------------------------------------------------
Centralized Desktop Delivery: Dell and VMware Reference Architecture
Simplifying enterprise desktop deployment and management using
Dell EqualLogic storage and VMware View: A highly scalable, end-to-end
client virtualization framework. Read more!
http://p.sf.net/sfu/dell-eql-dev2dev
_______________________________________________
Stripes-development mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/stripes-development

Reply via email to