don't throw 500s when trying to find() nonexistent pages

Author: drusepth

app/controllers/content_controller.rb

@@ -40,7 +40,8 @@ def index
   def show
     content_type = content_type_from_controller(self.class)
     return redirect_to root_path unless valid_content_types.map(&:name).include?(content_type.name)
-    @content = content_type.find(params[:id])
+    @content = content_type.find_by(id: params[:id])
+    return redirect_to(root_path, notice: "You don't have permission to view that content.") if @content.nil?
     @serialized_content = ContentSerializer.new(@content)
 
     return redirect_to(root_path) if @content.user.nil? # deleted user's content

app/controllers/users_controller.rb

@@ -4,7 +4,9 @@ def index
   end
 
   def show
-    @user    = User.find(params[:id])
+    @user    = User.find_by(id: params[:id])
+    return redirect_to(root_path, notice: 'That user does not exist.') if @user.nil?
+
     @content = @user.public_content.select { |type, list| list.any? }
     @tabs    = @content.keys
     @stream  = @user.content_change_events.order('updated_at desc').limit(100).group_by do |cce|