Hi,

I am busy working on a site which will have closed membership when it goes live, so I wish to disable Drupal ID logins, but from what I can tell it seems that it is not possible to disable that without disabling the whole Drupal module, which, in turn causes my site not to be listed in the Drupal Sites section.

Is this desired, or a bug, or am I just picky?

Regards,

Kobus

Comments

adrian’s picture

adrian commented

I understand the need, i have also had the need myself a few times. and have accomplished this by modifying the drupal.module file, and just commenting out everything from after the drupal_info() function.

killes@www.drop.org’s picture

killes@www.drop.org commented

A workaround has been offered. The core module will not be changed.

eveltman’s picture

eveltman commented

Shouldn't this be configurable ? I agree that a workaround has been offered, but the workaround can cause problems with upgrading. If you forget to remove the Drupal auth. part after upgrading, your site is open for drupal login IDs again.

praseodym’s picture

praseodym commented

It's still not possible to disabled Drupal-ID logins. I would recommend an option to disable this or simply not allowing Drupal-ID logins for closed registration sites.

Kobus’s picture

Kobus commented

I will try to scratch my own itch here and present the patch, if I can manage this.

Kobus

markus_petrux’s picture

markus_petrux commented

I believe, after this recently applied patch, it will now be possible.

HTH+

Kobus’s picture

Kobus commented

Thanks for the link! Not sure if this is already applied to drupal.module, but I'll check. I do, however, believe that this approach is WAY to elaborate for the simple functionality that is required. I believe the patch to fix this issue would be less than 2KB. I will have a crack at it and feed back here.

Regards,

Kobus

Kobus’s picture

Kobus commented
Assigned: Unassigned » Kobus
Status: Active » Fixed

I have read through the code of drupal.module supplied with beta 3 of Drupal 4.7, and this code is there:

  $form['services']['drupal_authentication_service'] = array(
    '#type' => 'radios',
    '#title' => t('Authentication service'),
    '#default_value' => variable_get('drupal_authentication_service', 0),
    '#options' => $options,
    '#description' => t('If enabled, your Drupal site will accept logins with the user names of other Drupal sites, and likewise provide authentication for users logging into other Drupal sites, based on their user accounts here.')
  );

and

function drupal_login($username, $password) {
  if (variable_get('drupal_authentication_service', 0)) {
    if ($user = user_load(array('name' => $username, 'pass' => $password, 'status' => 1))) {
      return $user->uid;
    }
    else {
      return 0;
    }
  }
}

So this is currently a non-issue. Closing the issue now.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.