railsbp/rails_best_practices — pull request #138 by RichardBradley (2012-11-03 18:32:36+00:00)
Title: Add check save return value review

I found this review quite hard to write, and it gives one or two false-positives on my current rails app, but I think this is an important best practice to check for since:
* it is an easy mistake to make
* you get no warnings from the framework about the error
* it is easy to lose data this way

I have put a couple of negative specs in showing the limitations of my approach. I would welcome any advice (or patches) to address these shortcomings.

http://rails-bestpractices.com/posts/703-check-the-return-value-of-save-otherwise-use-save