This section of code should be refactored into a method call, which is even implied by the comment at the top of the block. The comment is unnecessary if you use an intention revealing method name:

```php
$this->cleanupEnrollees($users);
```

Even better, refactor `setupUsersFromIdentity()` into two separate method calls wrapping the two main blocks of code in the method so that they can be easily understood and tested: 

``php
$identity = IdentityFactory::getIdentity();
$users = $identity->getAllUsers();
$this->cleanupEnrollees($users);
$this->updateEnrollees($users);
```