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
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

drewish - November 5, 2008 - 19:21

which version are you using? there aren't any tabs in the 2.x version.

#2

westwesterson - November 6, 2008 - 00:06
Version:6.x-2.x-dev» 6.x-2.0-beta2

beta 2

#3

drewish - November 6, 2008 - 00:59

then where are you seeing the links?

#4

westwesterson - November 6, 2008 - 18:08

http://example.com/node/536/translate

The links the go to the node edit pages from the translate page.

#5

drewish - November 6, 2008 - 19:17

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=de
http://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

westwesterson - November 7, 2008 - 00:44

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

drewish - November 7, 2008 - 00:49

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

drewish - November 17, 2008 - 08:44
Status:active» postponed (maintainer needs more info)

#9

drewish - April 16, 2009 - 19:41
Status:postponed (maintainer needs more info)» closed

i'm assuming you got it figured out due to the lack of feedback.

#10

ru.meta - September 2, 2009 - 12:16

I also have links like

node/540/edit?query=destination%3Dadmin%252Fcontent%252Ftranslation_overview_manage

My 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

ru.meta - September 2, 2009 - 12:18
Version:6.x-2.0-beta2» 6.x-2.2
Status:closed» needs review
 
 

Drupal is a registered trademark of Dries Buytaert.