Option to exclude blocked users from a role

There is too much overlap in the code. We should rewrite this using the Database API. Something like this would be much more efficient and transparent.

$query = db_select('users', 'u');

$query->addField('u', 'mail');

if (!$blocked) {
  $query->condition('u.status', 1, '=');
}

if (in_array(DRUPAL_AUTHENTICATED_RID, $roles)) {
  $query->condition('u.uid', 0, '>');
}
else {
  $query->join('users_roles', 'r', 'u.uid = r.uid');
  $query->condition('r.rid', $roles, 'IN');
  $query->distinct();
}

$result = $query->execute();

See the attached patch.

Committed!