I think we should use update_all instead: http://apidock.com/rails/v3.0.9/ActiveRecord/Relation/update_all - in particular for performance reasons -> one sql query instead of 2n queries + instantiation.

Also the timestamp at least for memberships is missin in the very initial records. So I would like to set it to the oldest value in the db now. That way all those really old memberships would actually be really old and give people long term membership permissions. This is less important though.