"Require a verification e-mail, but let users set their password directly on the registration form." option not working

Hey Tomas,

Do you have any other modules installed, like logintoboggan?

I guess we ran into the same issue. Described by the title that is.

Site replies
The 'fix' lies in changing

    // Some redundant checks for extra security ?
    $users = user_load_multiple(array($uid), array('status' => '0', 'access' => '0'));

into

    // Some redundant checks for extra security ?
    $users = user_load_multiple(array($uid), array('access' => '0'));

allows the link from the verification mail work instead of failing.

But 'status' needs to be 0, aka blocked and 'status' => '0', 'access' => '0' is blocked / never activated / used.

There's a test for this on the dev version, and the test is not failing.

Add a dpm($user) or print_r($user) on it to debug, the user needs to be disabled, else the user should not be able to use the link.

Also: try to upgrade to the latest dev (not 1.4-rc1), see if this fixes the problem.

The Test and Dev code .

I'm running 7.x-1.3 and was having issue with no registration emails being sent to the user. After finally down to disabling modules I identified this module as the culprit.

After uninstalling the module all registration emails were being sent.

Note... I have not tried the fix in #2 or the latest dev release.

@Rob C I'm not sure what our site had. But the accounts had 'status' == 1 already by config? or another module. Sorry :/

more facts about bug

I had this issue and the cause was a conflict with logintoboggan module. logintoboggan was setting the status to 1 in the submit handler.

The fix for this module would be to do it's submit handling before logintoboggan, however there could be further complications with that! It seems these modules do not play nicely together.

Hi All

I have same problem, please reply anybody if you have solution.

Thank you
Raushan Tiwari

Issue cleanup. Closing up. + We have a more recent issue with a concept patch. See the related issue.