This is ugly. We ripped 'access' TRUE for security reasons. So custom items have no access attribute any more. When a non cacheable (contextual) menu item is added, it won't override the one that's already there (ops) so it will walk up the menu tree until it finds an access . For node/nid items, it'll find node which means that it'll end up with TRUE. If there were no parents defined in the cacheable tree then you are hosed: you get access denied.

CommentFileSizeAuthor
#1 menu_29.patch699 byteschx
menu_28.patch698 byteschx

Comments

chx’s picture

StatusFileSize
new699 bytes

I forgot a $

chx’s picture

Status: Reviewed & tested by the community » Needs work

damn! this is not applicable for 4.6, this bug was introduced in 4.7 with the callback separation patch. But why then some 4.6 sites are broken? Back to drawing board.

dww’s picture

chx, any idea if this bug might be related to #55457? just curious what you think. i haven't had a chance to dig into that other one yet (it's a pretty rare bug, not critical), but i stumbled upon this issue and thought they sounded somewhat similar. let me know what you think.

thanks,
-derek

dww’s picture

argh, sorry that link is broken in my previous comment. damn preview not working on drupal.org after the upgrade to 4.7. :(

anyway, i'm talking about http://drupal.org/node/55457

magico’s picture

@chx: Any news about this?

magico’s picture

Status: Needs work » Closed (fixed)
magico’s picture

Version: 4.6.6 » 5.x-dev
Status: Closed (fixed) » Closed (duplicate)