Permalink
Browse files

Use scoped.first and scoped.last instead of find(:first, ...) and fin…

…d(:last, ...)
  • Loading branch information...
1 parent 1e7cf6c commit 1a4bbaf106f09c8a67c7e4da109a46449c06374f @jonleighton jonleighton committed Jan 24, 2011
Showing with 2 additions and 2 deletions.
  1. +2 −2 activerecord/lib/active_record/associations/association_collection.rb
@@ -39,7 +39,7 @@ def find(*args)
# Fetches the first one using SQL if possible.
def first(*args)
if fetch_first_or_last_using_find?(args)
- find(:first, *args)
+ scoped.first(*args)
else
load_target unless loaded?
args.shift if args.first.kind_of?(Hash) && args.first.empty?
@@ -50,7 +50,7 @@ def first(*args)
# Fetches the last one using SQL if possible.
def last(*args)
if fetch_first_or_last_using_find?(args)
- find(:last, *args)
+ scoped.last(*args)
else
load_target unless loaded?
@target.last(*args)

0 comments on commit 1a4bbaf

Please sign in to comment.