Email Validate Link Landing Page
Robert Castelo - August 4, 2008 - 17:12
| Project: | LoginToboggan |
| Version: | 5.x-1.3 |
| Component: | Code |
| Category: | feature request |
| Priority: | normal |
| Assigned: | Unassigned |
| Status: | closed |
Description
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.
#1
i'm confused -- isn't this what "Redirect path on Confirmation" does already??
#2
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.
#3
If that's what "Redirect path on Confirmation" is supposed to do I'll test further to see what's happening.
#4
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.
#5
i believe you're running into this case:
<?php
// 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...
#6
won't do this until another stable branch.
#7
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.
#8
Automatically closed -- issue fixed for 2 weeks with no activity.