This project is archived and is in readonly mode.

Changeset [8d337e9ec2e25007d557150dbe7557ab3c3bd05f] by Eloy Duran

October 3rd, 2008 @ 08:28 PM

Dynamic finders should use the ActiveRecord::Base::find method instead of ::find_initial, :find_last, and ::find_all.

This is so when people override ActiveRecord::Base::find, the new ::find method will also be invoked by the dynamic finders. Associations for instance do go through ::find, so this makes it more consistent. Also removed the unnecessary deprecation silence blocks.

Signed-off-by: Michael Koziarski michael@koziarski.com

[#1162 state:committed] http://github.com/rails/rails/co...

Committed by Eloy Duran

  • M activerecord/lib/active_record/base.rb
  • M activerecord/lib/active_record/dynamic_finder_match.rb
  • M activerecord/test/cases/finder_test.rb

Create your profile

Help contribute to this project by taking a few moments to create your personal profile. Create your profile »

<h2 style="font-size: 14px">Tickets have moved to Github</h2>

The new ticket tracker is available at <a href="https://github.com/rails/rails/issues">https://github.com/rails/rails/issues</a>