On Jul 10, 6:34 am, "Rick DeNatale" <[EMAIL PROTECTED]> wrote:
> In any event, I'm glad to see this come up.  Not having looked at the
> actually code in the patch, this looks like it might help address my current
> pet peeve about rails validations which is that the walkbacks you get when a
> validation throws an exception are pretty much worthless for determining
> WHICH validation failed.

That depends, what exactly made the backtraces suck before? I'm
guessing methods generated by Callbacks, but those should still show
the right file/line number (iirc). If not, then that's a separate fix
to ActiveSupport::Callbacks (and a simple one at that).

Although validations are no longer BASED on callbacks they still use
callbacks for conditional validation.

>
> Another benefit of what I'm reading is that it would make validations
> reflectable which would help in writing TDD/BDD assertions/matchers that a
> model did or did not validate certain things.

ABSOLUTELY it would! :)

>
> --
> Rick DeNatale
>
> My blog on Rubyhttp://talklikeaduck.denhaven2.com/
--~--~---------~--~----~------------~-------~--~----~
You received this message because you are subscribed to the Google Groups "Ruby 
on Rails: Core" group.
To post to this group, send email to rubyonrails-core@googlegroups.com
To unsubscribe from this group, send email to [EMAIL PROTECTED]
For more options, visit this group at 
http://groups.google.com/group/rubyonrails-core?hl=en
-~----------~----~----~----~------~----~------~--~---

Reply via email to