This can improve clarity, but it's ugly as hell IMO def make_person_an_outside_subscriber_if_all_accesses_revoked person.update_attribute(:outside_su

This can improve clarity, but it's ugly as hell IMO def make_person_an_outside_subscriber_if_all_accesses_revoked person.update_attribute(:outside_subscriber, true) if person.reload.accesses.blank? end I prefer camel case