Nobody Listens Anyway

Don’t forget to look at The Rails Way blog.
Move logic into your model. This lets you break functionality into more granular components and makes testing easier. Keep your controllers skinny. Jamis blogged about this principle here.
Use before filters to setup state before your controller runs. Use before_create in your models.
ActiveSupport helps [...]

The review is actually less painful than I would have thought. They are looking at parts of the code and giving specific feedback on code blocks. Here’s a quick summary:
‘and’ and ‘or’ are different than ‘||’ and ‘&&’. The second returns boolean values. The first returns strings. [...]