If your Drupal site is not at the root of you webserver, the "Redirect path on Confirmation: " will not work.

Suppose your have the following:
* drupal site url: http://localhost/my_drupal/
* Redirect on confirmation: node/3

After clicking an email confirmation link, and taken to the user edit page, the url will look like:
* http://localhost/my_drupal/user/31/edit?destination=/my_drupal/node/3

Note that the destination param should be "node/3" not "/my_drupal/node/3"

r1.123 started using drupal's url() function to build the destination parameter. I believe this is wrong.

The attached patch removes the use of url(), lets drupal_goto() handle the urlencode()ing of destination, and passes the fragment as an argument to drupal_goto().

IRC nick is Bentley if you wan to chat about this.

CommentFileSizeAuthor
logintoboggan.dest_.patch1.12 KBryan_courtnage
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

hunmonk’s picture

the problem with that patch is that it excludes query and fragment parts in the destiination.

however, i spent some time playing with things, and i'm not able to figure out how to get query and fragment parts included inside of another query part.

anybody have any thoughts?

mrmelson’s picture

I've run into this problem as well. I don't need query args & fragments ... so, I'm going to use this patch. Hopefully, a permanent fix will be found & included b4 the next release ;)

hunmonk’s picture

Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.