[jbossseam-issues] [JBoss JIRA] Commented: (JBSEAM-2245) Custom validator disables hibernate default validator

Pete Muir (JIRA) jira-events at lists.jboss.org
Fri Nov 23 08:26:18 EST 2007

    [ http://jira.jboss.com/jira/browse/JBSEAM-2245?page=comments#action_12388906 ] 
Pete Muir commented on JBSEAM-2245:

I guess we could add a flag like supplement="true" onto s:validateAll.  You could open a feature request for this.

> Custom validator disables hibernate default validator
> -----------------------------------------------------
>                 Key: JBSEAM-2245
>                 URL: http://jira.jboss.com/jira/browse/JBSEAM-2245
>             Project: JBoss Seam
>          Issue Type: Bug
>          Components: JSF
>    Affects Versions: 2.0.0.GA
>            Reporter: dapeng wang
>         Assigned To: Pete Muir
>             Fix For: 2.0.1.GA
> The default hibernate validator doesn't work anymore, as soon as a custom validator is added to the component, though the text field is put inside a validateAll tag. 
> In the class ValidateAllRendererBase following code is found, which is buggy in my opinion:
> if ( evh.getValidators().length==0 && evh.getValidators().length == 0 )
>             {
>                evh.addValidator( new ModelValidator() );
>             } 	
> First of all, I don't understand, why you check the same condition twice. Second, I have no clue, why you only add the ModelValidator, if here is no other validator. I would exactly like to combine the hibernate validator with my custom one. 
> Pleas remove the if check. I would also like to help and fix this bug.

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


More information about the seam-issues mailing list