I use this module along with LoginToboggan. By turning on email verification in LT users are assigned a temp role until their account is verified. When a user changes their email address before verifying it from LT they will need to verify their email twice. Once for this module, another for LT. Is there a way to verify LT account as well when verifying this one?

Comments

shadysamir’s picture

shadysamir commented
Title: Integrate with LoginToboggan » Integrate with LoginToBoggan

I wrote the following module to integrate with LoginToboggan using hook_email_confirm:

function mymodule_email_confirm($op, $uid, $old_email, $new_email){
  switch ($op) {
    case 'email confirmation':
      $account = user_load($uid);
      _logintoboggan_process_validation($account);
      break;

    default:
      break;
  }
}
shadysamir’s picture

shadysamir commented
Title: Integrate with LoginToBoggan » Integrate with LoginToboggan
BenK’s picture

BenK commented
Title: Integrate with LoginToBoggan » Integrate with LoginToboggan

Subscribing

jaydub’s picture

jaydub commented
Version: 6.x-1.x-dev » 7.x-1.x-dev

As @shadysamir noted this can be done by implementing the relevant hook when the email is confirmed. Of course as is often the issue the question is whether this should be done by the LoginToboggan module or by the email confirm module.

I'm guessing that LoginToboggan won't take the time to implement this although that's my desired solution. In lieu of that I suppose I can have the implementation be in email confirm.

I will look into this for d7 first - no timetable on d6.

jaydub’s picture

jaydub commented
Assigned: Unassigned » jaydub
jaydub’s picture

jaydub commented

@shadysamir Can you go over with me the specific settings in LoginToboggan I should set when addressing this issue? I am speaking of the 'Set password' checkbox on the LB settings page as well as what core user settings should be set for this situation.

greggles’s picture

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

I think this is a better status. IMO this issue could be skipped.

greggles’s picture

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

Closing based on lack of feedback from 5 years ago when jaydub asked in #6 how this should work. If anyone adds details, please do reopen.