Does this module prevent people from registering if they are not listed?

I thought this module would keep people from being able to register if not pre-approved?

No, the module doesn't prevent users from registering. It's only intended to assign custom roles to users that match a "pre-approved" email address or domain.

Your idea is a good one though and might make a nice feature request, but I don't personally have the bandwidth to facilitate such a request unless it's paid work.

We implemented this with a small custom module (in 7.x, not 6.x). It works something like this:

function MYMODULE_form_user_register_form_alter(&$form, &$form_state, $form_id) {
  $form['#validate'][] = 'MYMODULE_validate_register_form';
}

function MYMODULE_validate_register_form(&$form, &$form_state) {

  $pattern = db_select('register_preapproved', 'rp')
    ->fields('rp')
    ->condition(db_or()
      ->condition('mail', $form_state['values']['mail'])
      ->condition('mail', strstr($form_state['values']['mail'], '@'))
    )
    ->execute()
    ->fetchAssoc();

  if (empty($pattern)) {
    $message = variable_get('MYMODULE_not_preapproved_message', t('Your email address has not been pre-approved for use on %site_name.'));
    form_set_error('mail', t($message, array('%site_name' => variable_get('site_name', 'Drupal'))));
  }
}

I'm not sure if that's worth a patch or not, maybe as an optional submodule to this one?

Hey Adam,

That looks like a simple elegant solution to OP's request--thanks! Ideally it should be a configurable option (which would be easy to implement).

The 7.x version of this module is sorely in need of some attention (i.e. there's a couple of other patches in the issue queue.) I will ping the maintainer (KeyboardCowboy) of the 7.x branch and see if he's got bandwidth to work on this otherwise I will carve out some time to do it myself.

John

Added to in 7.x-1.0 release. Thank you!