I just installed 1.x-dev for the bug with devel. But it seems there is a new fearture of switching user without option to disable. It makes the menu so ugly because you don't how strange the user name is! So I've to look through the code, and find it bundled many new code with devel. I don't think it's a good design for a module under Administration, and devel has done all things well with itself. Do we have to be a module of devel menu?

CommentFileSizeAuthor
#6 before.jpg9.38 KBvordude
#6 after.jpg17.08 KBvordude
#5 433016_switch_user-4.patch616 bytesultimateboy

Comments

upupax’s picture

I didn't realize it was a new feature.. I've all my users displayed in the menu admin between the site links.
I think its a bit "messy". I mean, I think it could be useful for someone, but IMHO they should be in a separate dropdown menu so that they don't stretch the menu structure.

ultimateboy’s picture

Title: Remove switch user items » Switch user items have no parent
Category: support » bug

This now appears to be a bug. admin_menu usually gracefully integrates with devel's swtich user by placing the switch user links as children of the logout link. However, it appears that what the previous two posters have reported, is indeed an issue.

I cannot pin-point the real issue at the moment, but all of a sudden, I began to see similar issues. I attempted a 'Wipe and rebuild', a rebuilding of the menu system, and a clearing of the cache, all unsuccessful in reverting the behavior back to how it should be.

The image below is a sample of what I am seeing. In this case the email addresses showing are actually the user names.
Only local images are allowed.

s.daniel’s picture

Confirming the error.
This bug appears on two (dev and live) sites out of 6 which run from the same codebase.
Disabled all modules at the dev site and deleted and reinstalled admin menu and devel - no success.

ultimateboy’s picture

Status: Active » Needs review
StatusFileSize
new616 bytes

'parent_item' was incorrectly set. Apply patch, wipe and rebuild admin menu.

vordude’s picture

StatusFileSize
new17.08 KB
new9.38 KB

Installed Admin Menu 6.x-1.x-dev and Devel 6.x-1.16 on a fresh install. I Confirmed previous issue with the usernames showing up as first level menu items. (as shown in the attachment labeled "before")

I Applied the Patch successfully, wiped the admin menu, and then cleared the cache.

Then the menu appeared to work properly, with the "user switcher list" dropping down "Log out (username)" button (as shown in the attached image labeled "after")

When I try to switch between users, the "?destination=" portion seems to get cached. The first time I take a look at it, to see if it works, it's fine, but I browse around a bit, and attempt to switch users, and it sends back to the destination where I first took attempted switching users. (If that made any sense)

I clear the cache, and it seems to "catch up"

I don't know if this is new behavior or an old bug, or something strange I did.

upupax’s picture

patch works fine for me!

sun’s picture

Status: Needs review » Fixed

Sorry, I copied this code from 3.x and did not change the path. Thanks, ulimateboy!

But, by the way, if you're using a development snapshot, I'd highly recommend to use 3.x-dev... ;)

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.

aaronchristian’s picture

I tried the patch above only to find out that I was using the 6x.-1.3 admin menu... DUH! But anyways,

I am having the exact same issue with the Admin Menu 6x-1.3 release combined with the Devel 6.x-1.16 version. I downgraded my admin menu's 6x-1.3 official release to the Development snapshot 6.x-1.x-dev. And everything was fixed after this.

If you are having the same issue, I'd suggest downgrading your admin menu and waiting for an appropriate patch or wait until this bug (if its a bug & others are experiencing it too), is fixed.