DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG 
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
<http://issues.apache.org/bugzilla/show_bug.cgi?id=29736>.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND 
INSERTED IN THE BUG DATABASE.

http://issues.apache.org/bugzilla/show_bug.cgi?id=29736

ValidatorForm.validate should be able to throw ValidatorException

[EMAIL PROTECTED] changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
         Resolution|                            |WONTFIX
            Summary|ValidatorForm.validate      |ValidatorForm.validate
                   |should be able to throw     |should be able to throw
                   |ValidatorException          |ValidatorException



------- Additional Comments From [EMAIL PROTECTED]  2004-06-22 14:58 -------
ValidatorException is a checked exception, which means it would need to be added to 
the interface for 
ActionForm, breaking backwards compatibility with all existing ActionForm 
implementations, many of 
which don't even use Commons Validator.

Arguably, we could create o.a.struts.validator.ValidatorException as a subclass of 
RuntimeExtension 
and throw that, but still, it would disrupt a lot of existing code to start throwing 
exceptions someplace 
where they were previously being caught.

In the meantime, you could throw a different runtime exception from your custom 
plugin, or do that 
part of the validation in the action instead of in form.validate().

We couldn't fix this without a suggestion for a less disruptive way to handle it than 
changing the 
signature of ActionForm.validate()

---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to