Ben, Good point. It is conceivable that code relies on it working this way... though I would guess the converse is more likely... in that people probably aren't aware of this outcome and may have a potential bug in their validation (assuming the converter outputs null).
Not a worry though however on that note... something that comes up all too often for new users of Stripes is... - Changing the default population strategy to use BeanFirstPopulationStrategy If you concur we should write up a JIRA ticket and tag it for 1.6 ... b/c if not 1.6 then when? Cheers, --Nikolaos Ben Gunter (JIRA) wrote: > [ > http://www.stripesframework.org/jira/browse/STS-752?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12065#action_12065 > ] > > Ben Gunter commented on STS-752: > -------------------------------- > > It has behaved this way for years. I don't want to alter this kind of > behavior for a minor release. > > >> @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. >> > > ------------------------------------------------------------------------------ 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
