Eloy Duran
2008-Oct-03 08:18 UTC
For review: Dynamic finders should go through AR::Base::find
Hi, I have created a patch which makes the dynamic finder methods use AR::Base::find with :first, :last, and :all instead of :find_initial, :find_last, and :find_all. This is because atm when one overloads ::find it doesn''t apply to queries performed by dynamic finders. Associations, for instance, do go through ::find, so this keeps it more consistent. http://rails.lighthouseapp.com/projects/8994-ruby-on-rails/tickets/1162-dynamic-finders-should-go-through-arbasefind Cheers, Eloy Duran --~--~---------~--~----~------------~-------~--~----~ 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 rubyonrails-core+unsubscribe@googlegroups.com For more options, visit this group at http://groups.google.com/group/rubyonrails-core?hl=en -~----------~----~----~----~------~----~------~--~---