No need to use ||= on the first assignment
update_attribute -> update!