Just a random thought.
A lot of people who use rails do a object.save, then check if it's true or not to determine what to do next in a controller. I find that to be verbose and hides the actual save command inside an if statement.
def update
if @page.update_attributes params[:page]
redirect_to admin_pages_path
else
render :edit
end
end
I prefer stuff like this:
def update
@page.update_attributes! params[:page]
redirect_to admin_pages_path
rescue ActiveRecord::RecordInvalid
render :edit
end
I think it's more clear that to render :edit is an exception and should not be the norm.