Index: logintoboggan.install =================================================================== RCS file: /Users/Shared/drupal_repos_rsync/contributions/modules/logintoboggan/logintoboggan.install,v retrieving revision 1.14 diff -u -F '^f' -u -F '^f' -r1.14 logintoboggan.install --- logintoboggan.install 8 Mar 2008 23:02:15 -0000 1.14 +++ logintoboggan.install 10 Mar 2009 23:59:49 -0000 @@ -150,6 +150,12 @@ function logintoboggan_update_6000() { return array(); } +// Alert admins to change in 'Redirect path on Confirmation' setting. +function logintoboggan_update_6001() { + drupal_set_message(t("Please note that the behavior of LoginToboggan's Redirect path on Confirmation setting has changed in the case where users are NOT setting their own password on registration. The previous behavior was to automatically redirect the user to user/%uid/edit, with a final destination of the Redirect path on Confirmation setting. This behavior has been removed. To replicate it, the path must be explicitly declared now. Ex: to have the user visit their edit page, with a final destination of node/foo, you would use user/%uid/edit?destination=node/foo.")); + return array(); +} + /** * Implementation of hook_uninstall(). */ Index: logintoboggan.module =================================================================== RCS file: /Users/Shared/drupal_repos_rsync/contributions/modules/logintoboggan/logintoboggan.module,v retrieving revision 1.133.2.7 diff -u -F '^f' -u -F '^f' -r1.133.2.7 logintoboggan.module --- logintoboggan.module 9 Dec 2008 23:53:47 -0000 1.133.2.7 +++ logintoboggan.module 10 Mar 2009 23:41:43 -0000 @@ -704,7 +704,7 @@ function logintoboggan_main_settings(&$f '#type' => 'textfield', '#title' => t('Redirect path on Confirmation'), '#default_value' => variable_get('logintoboggan_redirect_on_confirm', ''), - '#description' => t('Normally, after a user confirms their new account, they will be taken to their user page. Leave this setting blank if you wish to keep the default behavior. If you wish the user to go to a page of your choosing, then enter the path for it here. For instance, you may redirect them to a static page such as node/35, or to the <front> page. You may also use %uid as a variable, and the user\'s user ID will be substituted in the path.'), + '#description' => t('Normally, after a user confirms their new account, they will be taken to their user page. Leave this setting blank if you wish to keep the default behavior. If you wish the user to go to a page of your choosing, then enter the path for it here. For instance, you may redirect them to a static page such as node/35, or to the <front> page. You may also use %uid as a variable, and the user\'s user ID will be substituted in the path. In the case where users are not creating their own passwords, it is suggested to use user/%uid/edit here, so the user may set their password immediately after validating their account.'), ); $form['other'] = array('#type' => 'fieldset', @@ -965,14 +965,10 @@ function logintoboggan_process_login($ac user_module_invoke('login', $edit, $user); // 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. + // own password, show a user message letting them know to change their password. 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); } if ($redirect['path'] != '') {