since we're using $may_cache incorrectly in hook_menu(), the links aren't showing up for this module in the admin/by-module page.

CommentFileSizeAuthor
project_issue_hook_menu_may_cache_5x.patch.txt2.81 KBdww

Comments

merlinofchaos’s picture

merlinofchaos commented

The patch looks ok. It appears as though it may be leaving an $access = line hanging around, but there may also be other items past it that I can't see without actually applying the patch.

dww’s picture

dww
we/he/they
commented
Status: Needs review » Fixed

thanks, yeah, that $access was still being used, it's just where the patch ended. thanks for looking carefully. committed to HEAD.

Anonymous’s picture

(not verified) commented
Status: Fixed » Closed (fixed)