In function custom_email_registration_name_submit(), the assignment of $form_state['values']['pass'] returns NULL as that variable is no longer set. The result is that user_authenticate() fails, and the new user is not logged in.

I believe this should be:

'pass' => $account->password,

Comments

greggles’s picture

greggles
he/him
Primary language English
Location Denver, Colorado, USA
commented
Status: Active » Postponed (maintainer needs more info)

If you pull the password from the loaded user object then it's guaranteed to always match. I wonder if that's a security issue or not.

Can you confirm this is still a problem with the 6.x-1.x-dev on a fresh install? It seems surprising to me that this is a module-wide bug and nobody else has reported this issue since May of 2011. My guess is that there is a bad interaction with another module OR some site specific setting that is a bit odd.

dwightaspinwall’s picture

dwightaspinwall commented
Status: Postponed (maintainer needs more info) » Closed (cannot reproduce)

I tested on a fresh install with no problems. Also tested against our site with no problems. We probably had some bad form_alter code on user_register or perhaps it was another contrib module that's been fixed. Thanks for following up.

greggles’s picture

greggles
he/him
Primary language English
Location Denver, Colorado, USA
commented

Phew, glad to hear this was a temporary bug.