Permalink
Browse files

removes unnecessary selfs, and mentions that first! and last! take no…

… arguments in their API docs
  • Loading branch information...
fxn committed Mar 25, 2011
1 parent 6383f67 commit 9d9b873b9501f5fc90c0b6a4a550c5e29fc742a5
Showing with 6 additions and 4 deletions.
  1. +6 −4 activerecord/lib/active_record/relation/finder_methods.rb
@@ -123,9 +123,10 @@ def first(*args)
end
end
- # Same as #first but raises RecordNotFound if no record is returned
+ # Same as +first+ but raises <tt>ActiveRecord::RecordNotFound</tt> if no record
+ # is found. Note that <tt>first!</tt> accepts no arguments.
def first!
- self.first or raise RecordNotFound
+ first or raise RecordNotFound
end
# A convenience wrapper for <tt>find(:last, *args)</tt>. You can pass in all the
@@ -142,9 +143,10 @@ def last(*args)
end
end
- # Same as #last but raises RecordNotFound if no record is returned
+ # Same as +last+ but raises <tt>ActiveRecord::RecordNotFound</tt> if no record
+ # is found. Note that <tt>last!</tt> accepts no arguments.
def last!
- self.last or raise RecordNotFound
+ last or raise RecordNotFound
end
# A convenience wrapper for <tt>find(:all, *args)</tt>. You can pass in all the

0 comments on commit 9d9b873

Please sign in to comment.