Subject says almost all. The result of the function is set as 'destination' parameter for the target url of forms. If the form is submitted and form input has been processed, Drupal will drupal_goto() to the 'destination' URL. Because drupal_goto() translates the URL to alias form, the URL passed to drupal_goto should be in source form.
At least, that's what I think ;-) I definitely don't know many intimate details of Drupal's inner functioning, so I could be wrong. I've posted my solution in http://drupal.org/node/78356
| Comment | File | Size | Author |
|---|---|---|---|
| #5 | common.inc.patch_4.txt | 631 bytes | eveltman |
Comments
Comment #1
drummThere isn't a patch file attached here.
Comment #2
eveltman commentedSorry for not making a patch file, I'm not too familiar with that tool and didn't have the time to make myself familiar with it. But I placed the fixed code on http://drupal.org/node/78356. That's why I thought 'patch (code needs review)' was an appropriate status for the issue.
Comment #3
eveltman commentedChanging status back to 'patch' again. See link in the original description. The fixed function is there.
Comment #4
stevenpatzThere is no patch to review.
Comment #5
eveltman commentedOK, I ran into it again with 4.7.4. Now I had a little more time, so I created a real patch file. Hope it's OK now.
Comment #6
killes@www.drop.org commentedmoving
Comment #7
Steven commentedThis patch does expose a non-clean URL, but as its only used for form submission, it's acceptable. Internally, we should never use aliases in drupal_goto() calls.
Tested and committed to HEAD.
Comment #8
killes@www.drop.org commentedapplied to 4.7
Comment #9
(not verified) commented