The devel related switch user available from admin_menu (right side, logout sub-links) redirects to the wrong destination.

I fixed it at line 292 of admin_menu.module:
http://drupalcode.org/project/admin_menu.git/blob/6bd5f95e8f25bb3de9dd8a...

Replace

    if ($item['localized_options']['query'] == 'destination') {

with

    parse_str($item['localized_options']['query'], $out);
    if (isset($out['destination'])) {

since the query may hold more than just the "destination" string, for instance, it could be "destination=node/123".

Same applies to latest 6.x-1.x dev version.

CommentFileSizeAuthor
#2 destination_query-1393222.patch662 bytesRobin Millette

Comments

sun’s picture

sun
Primary language German
Location Karlsruhe
commented
Assigned: Unassigned » pwolanin

Assigning to @pwolanin, as he fast-tracked those CSRF changes into 6.x-1.x, IIRC.

Robin Millette’s picture

Robin Millette commented
Status: Active » Needs review
StatusFileSize
new destination_query-1393222.patch662 bytes

Attached is a proper patch, based on 6.x-1.x branch.

truls1502’s picture

truls1502
he/him
Primary language Norwegian Bokmål
commented
Issue summary: View changes
Status: Needs review » Closed (outdated)

It seems it is outdated because Drupal 6 is EOL (End-of-life) and will no longer be supported.

In case if you or anyone is still facing on Drupal 7/8, please to reopen it, change the correct version and provide with more information and a screenshot which might help us to troubleshoot. :)