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.
Comment | File | Size | Author |
---|---|---|---|
logintoboggan.dest_.patch | 1.12 KB | ryan_courtnage |
Comments
Comment #1
hunmonk CreditAttribution: hunmonk commentedthe 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?
Comment #2
mrmelson CreditAttribution: mrmelson commentedI'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 ;)
Comment #3
hunmonk CreditAttribution: hunmonk commentedthis should be fixed by #291006: Email Validate Link Landing Page