This is a very useful function. As with the comment below, I would reference `roles` instead of `role_ids`. (Note that you'll then need to change references to the method-local variable `roles` to `self.roles`.)

I think I prefer the method signature:

```ruby
def set_organization_roles
```

Since the memberships are opaque to the caller, I think it's better for the method not to reference them.