Email Validate Link Landing Page [#291006]

Logintoboggan currently has a feature to specify the redirect to be used once the user has verified email and been taken to the account edit page.

Sometimes the account edit page is not the best landing page though - there may be info that it's important to try and collect on other profile pages, and we may not really care too much if they change their password or not, or any of the other options.

So it would be great to have a way to specify the landing page after the email validate link has been clicked.

If there's interest in this idea I'd be happy to code it.

Comment	File	Size	Author
#7	lt.patch	4.1 KB	hunmonk

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Comment #1

hunmonk CreditAttribution: hunmonk commented 6 August 2008 at 15:02

Status:

Active

» Postponed (maintainer needs more info)

i'm confused -- isn't this what "Redirect path on Confirmation" does already??

Comment #2

Robert Castelo CreditAttribution: Robert Castelo commented 6 August 2008 at 20:38

I thought so too, but the behaviour I get is that it goes to the account edit page and sets the redirect for the next page after that.

Comment #3

Robert Castelo CreditAttribution: Robert Castelo commented 6 August 2008 at 21:16

If that's what "Redirect path on Confirmation" is supposed to do I'll test further to see what's happening.

Comment #4

2c CreditAttribution: 2c commented 8 September 2008 at 11:33

Subscribe. I'm getting this too. The user clicks on the one time validation link and they are taken to the page to edit their password. Then, when they click the submit button to change their password they are taken to the $destination.

Comment #5

hunmonk CreditAttribution: hunmonk commented 15 September 2008 at 01:59

i believe you're running into this case:

  // In the special case where a user is validating but they did not create their
  // own password, redirect them to the user edit page, with a final destination
  // of the confirmation page, if it exists.
  if (variable_get('user_email_verification', TRUE)) {
    watchdog('user', 'User %name used one-time login link at time %timestamp.', array('%name' => $user->name, '%timestamp' => time()));
    drupal_set_message(t('You have just used your one-time login link. It is no longer necessary to use this link to login. Please change your password.'));

    $destination = $redirect['path'] ? 'destination='. url($redirect['path'], array('query' => $redirect['query'], 'fragment' => $redirect['fragment'])) : NULL;
    drupal_goto('user/'. $user->uid .'/edit', $destination);
  }

we *may* be able to clobber this code, as i believe the support for query strings in the confirmation setting is supported now, so this could be done by choice with a 'user/%uid/edit?destination=page' approach.

i'd hate to tear that out in a stable branch, though -- i do have a 6.x-2.x branch possibility...

Comment #6

hunmonk CreditAttribution: hunmonk commented 11 October 2008 at 00:07

Status:

Postponed (maintainer needs more info)

» Postponed

won't do this until another stable branch.

Comment #7

hunmonk CreditAttribution: hunmonk commented 11 March 2009 at 00:18

Status:

Postponed

» Fixed

File	Size
lt.patch	4.1 KB

this 'feature' has been causing problems over at #346713: Redirect path on Confirmation is broken and #372884: Validation email links always result in Access Denied message, so i've decided to go ahead and remove it in the 6.x-1.x branch -- see attached patch. i won't mess with this in 5.x, so an upgrade will be necessary to benefit from this enhancement.