When overloading an ActiveRecord setter, I've often done this:

   1  def name=(new_name)
   2    @old_name = name
   3    attributes['name'] = new_name
   4  end

However, this doesn't always seem to work (changes to attributes don't seem to stick). Recently I've started doing this:

   1  def name=(new_name)
   2    @old_name = name
   3    super
   4  end

Which seems cleaner anyway. I can't seem to find anyone talking about the correct convention - anyone know the right answer here?