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

Nikolaos commented on STS-752:
------------------------------

Your correct Freddy.  The type converter has the opportunity to throw an error 
but I don't think that was the issue.

The question is how far does "@Validate(required=true)" take you?  If it goes 
as far as the user having provided some non-empty input as Oscar points out 
then everything is working as expected.  And now that I think about this again 
I totally agree with Oscar's assessment.

So... Aaron... what do you think?

--Nikolaos

> @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