Much better to use hook_menu_alter() because it works only on menu rebuild but hook_menu_link_alter() executed on each request

CommentFileSizeAuthor
#4 masq-menu.png13.06 KBandypost
#4 masq-menu1.png10.73 KBandypost
masquerade-menu.patch1.47 KBandypost

Comments

podarok’s picture

podarok
🇺🇦 he/him/his
Primary language Ukrainian
Location Ukraine
commented

works for me

tested on D7-RC2 with already applied patches
#1004004: Fix user profile link + #1004002: Block is empty

alberto56’s picture

alberto56 commented

The patch doesn't fix #1005198: No "Switch back" link in D7 version, which is said to be a duplicate of the current issue.

andypost’s picture

andypost
he/him
Primary language Russian
commented
Status: Needs review » Reviewed & tested by the community

@alberto56 I've tested unswitch with & without this patch - both cases works fine
Do you clear cache or are you using some cache backend?

andypost’s picture

andypost
he/him
Primary language Russian
commented
StatusFileSize
new masq-menu1.png10.73 KB
new masq-menu.png13.06 KB

Screens attached

alberto56’s picture

alberto56 commented

@andypost thanks for your message. I'm confirming that you're right: both "switch back" links (in the navigation menu and the masquerade block) work without this patch on a clean install. I'll have to track down what is causing the "switch back" link to not appear on my test site and, if necessary, I'll reopen #1005198: No "Switch back" link in D7 version.

Cheers,

Albert.

deekayen’s picture

deekayen commented
Status: Reviewed & tested by the community » Closed (duplicate)

Looks like this was rolled into #1006668: General code cleanup and wish to make D7rc1 release