drupal return destination does not behave as expected.
westwesterson - November 5, 2008 - 18:20
| Project: | Translation Overview |
| Version: | 6.x-2.2 |
| Component: | Code |
| Category: | bug report |
| Priority: | normal |
| Assigned: | Unassigned |
| Status: | needs review |
Jump to:
Description
There is something wrong with the implementation of drupal_get_destination in this module as accessed from the translation overview tab.
I think it has to do with drupal_get_destination being called incorrectly because the function returns destination=url already, but the translation_overview code adds an extra destination=. So URLS look like destination=destination%39node%39123 which is wrong.

#1
which version are you using? there aren't any tabs in the 2.x version.
#2
beta 2
#3
then where are you seeing the links?
#4
http://example.com/node/536/translateThe links the go to the node edit pages from the translate page.
#5
oh, gotcha, i understand what you mean now--urls are always helpful ;)
but that said, i'm not seeing the behavior you're describing... i'm seeing links like:
http://nm/?q=en/node/add/image&destination=node/2264/translate&translation=2264&language=dehttp://nm/en/node/add/image?destination=node/2264/translate&translation=2264&language=de
is there a particular way that you're getting to the node's translation tab?
#6
no, nothing in particular...
full url looks something like this:
http://nm/en/node/738/edit?0=destination=destination%3Dnode%252F777%252F...
clean urls turned on
the 0 is weird...
#7
i'm not really sure what to tell you... do you have any other modules installed that would be rewriting links? if you'd like to message me an account and password i can login and take a look.
#8
#9
i'm assuming you got it figured out due to the lack of feedback.
#10
I also have links like
node/540/edit?query=destination%3Dadmin%252Fcontent%252Ftranslation_overview_manageMy version is 6.x-2.2
I found that if I replace
$link['options']['query'] = array('destination' => drupal_get_destination());with
$link['options']['query'] = drupal_get_destination();Links work well
#11