On 11/2/05, Tim Lesher <[EMAIL PROTECTED]> wrote:
> > Right now, it's just a boolean. You can get the keys with errors by
> > grabbing cherrypy.request.form_errors.keys()
>
> Hmm... if you can always get the form errors from this attribute,
> isn't has_errors redundant? DRY, and all...

That's a good point. If we deprecate validation_error, then the method
can just be called directly and you can check form_errors to see if
there's a problem. I think this also means that an invalid exception
would never be raised.

It simplifies the code, but it does also eliminate a visual reminder
to check for errors. Maybe that's not so bad, though.

Kevin

Reply via email to