This project is archived and is in readonly mode.

add first_bang to AR

#6613

Before scopes, controller methods did this:

@user = User.find(params[:id])

Now with scopes you can do

@user = User.where(:id => params[:id]).first

However that doesn't raise an exception when the record is not found. With this patch, you can do this:

@user = User.where(:id => params[:id]).first!

And that raises the RecordNotFound exception, as you'd expect.

Reported by Josh Susser · March 24th, 2011 @ 05:03 AM

Andrew White
- State changed from new to open
- Milestone set to 3.1
- Assigned user changed from Aaron Patterson to Andrew White
- Importance changed from to Low
Josh, can you add last! as well to the patch and I'll apply it

March 24th, 2011 @ 11:17 AM
Josh Susser
OK, added last! too. (Aaron, you were right!)
- first_bang.diff 2.62 KB
March 24th, 2011 @ 04:33 PM
Josh Susser
By the way, the simplistic example given in the original ticket above isn't the motivating use case. That would be using an association or scope to find a record.
```
def show
  @latest_update = current_user.updates.recent.first!
end
```
March 24th, 2011 @ 04:45 PM
Aaron Patterson
- State changed from open to committed
Applied and pushed. Thanks.

March 24th, 2011 @ 04:55 PM
Josh Susser
Ugh, typo in comments. How lame is that? Here's the fix.
- first_bang_typo_fix.diff 1.16 KB
March 25th, 2011 @ 04:46 PM