Handling ActiveRecord::RecordNotFound in Ruby on Rails

Question

In my edit action, I have

@item = current_user.shop.items.find(params[:id])

So that the user can only edit items that belong to their shop. If they try to edit an item that does not belong to their shop, then they get an ActiveRecord::RecordNotFound error.

What is the best way of handling this error in situations like this? should i raise an exception? should i redirect somewhere and set the flash (if so, how do i do that), should i just leave it as is? Any advice is appreciated.

Thanks

Blue Smith · Answer 1 · 2013-06-06T02:42:47.537

22

+1 for the solution of @Koraktor. If you want to avoid ActiveRecord::RecordNotFound, you can use find_by method instead of find.

@item = current_user.shop.items.find_by_id(params[:id])
if @item.nil?
  flash[:error] = "Item not found"
else
  # Process the @item...
end

edited Jun 06 '13 at 02:42

answered Aug 30 '12 at 03:47

Blue Smith

8,580
2
28
33

Koraktor · Accepted Answer · 2011-07-25T11:09:58.727

22

Add something like the following to your controller:

rescue_from ActiveRecord::RecordNotFound do
  flash[:notice] = 'The object you tried to access does not exist'
  render :not_found   # or e.g. redirect_to :action => :index
end

edited Jul 25 '11 at 11:09

answered Jul 25 '11 at 10:32

Koraktor

41,357
10
69
99

score 5 · Answer 3 · edited May 23 '17 at 10:27

Additionally you should set the http status while rendering the template :

rescue_from ActiveRecord::RecordNotFound do
  render :not_found, :status => :not_found
end

Check the Rails guide (section 2.2.13.4) for the status list.

As stated somewhere else (for example here ) you should never make a redirection when sending a 40x status, only 30x statuses should allow redirecting. So you may encounter a weird "You are being redirected" page whenever trying to make a redirect with a :not_found status.

Handling ActiveRecord::RecordNotFound in Ruby on Rails

3 Answers3