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

[Pete Muir (JIRA)]

     [ http://jira.jboss.com/jira/browse/JBSEAM-2245?page=all ]

Pete Muir closed JBSEAM-2245.
-----------------------------

    Resolution: Won't Fix
      Assignee: Pete Muir

You can add both your custom validator and the hibernate validator like this

<h:inputXXX>
   <s:validate />
   <my:validate />
</h:inputXXX>

> 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