TEST_CODEOWNERS/rails
3
0
Fork 0
Code Issues Pull Requests 1 Actions Packages Projects Releases Wiki Activity

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

Browse Source
This commit is contained in:
Jon Leighton 2011-01-24 22:14:32 +00:00
parent 1e7cf6c7e9
commit 1a4bbaf106
1 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
activerecord/lib/active_record/associations/association_collection.rb
View File

@ -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)