Replace hook_menu_link_alter() with hook_menu_alter() [#1004014]

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

Comment	File	Size	Author
#4	masq-menu.png	13.06 KB	andypost
#4	masq-menu1.png	10.73 KB	andypost
	masquerade-menu.patch	1.47 KB	andypost

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Comment #1

podarok

🇺🇦 he/him/his

Ukrainian

Ukraine

CreditAttribution: podarok commented 21 December 2010 at 09:54

works for me

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

Comment #2

alberto56 CreditAttribution: alberto56 commented 23 December 2010 at 01:39

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

Comment #3

andypost

he/him

Russian

CreditAttribution: andypost commented 23 December 2010 at 02:48

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?

File	Size
masq-menu1.png	10.73 KB
masq-menu.png	13.06 KB

Comment #5

alberto56 CreditAttribution: alberto56 commented 23 December 2010 at 14:37

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