Just a suggestion. Feel free to nack.. 
I 'd like ```self.save!``` to complain if there are errors
I 'd like method to be called ```clear_existing_ldap_roles!``` to reflect the fact that it is changing the state of the object.