''Note: if you have an account with one of our affiliates Drupal, you may login now instead of registering.'' appears even if that feature is disabled.

CommentFileSizeAuthor
user_78.patch962 byteshe_who_shall_not_be_named

Comments

he_who_shall_not_be_named’s picture

Status: Active » Needs review

Changed status

drumm’s picture

Version: 5.1 » 6.x-dev
Component: user.module » drupal.module
Status: Needs review » Needs work

Although this method is used one other place in user module, it it not good. The user module should not have to worry about accessing any other module's variables, including the drupal module. The drupal module should define drupal_auth() and related files in a separately included file, which is only included when the authentication service is enabled.

AjK’s picture

Project: Drupal core » Site Network
Version: 6.x-dev »
Component: drupal.module » Code
brmassa’s picture

Version: » 6.x-1.x-dev
Assigned: Unassigned » brmassa
Status: Needs work » Fixed

Guys,

Drupal 6 now relies on Form API for managing it. Fixed on CVS.

regards,

massa

Anonymous’s picture

Status: Fixed » Closed (fixed)

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