On line 230 the path value is not set because the 'path'=> is missing. This caused a warning message when the menu module is enabled and a new menu is created. And I'm sure that the calender link was not working either. Patch is attached.

CommentFileSizeAuthor
event_6.patch704 bytesjsloan

Comments

anj’s picture

anj commented
Title: Broken MENU declaration » Broken MENU declaration in event_menu hook
Priority: Normal » Critical

Please tell me if I'm doing the wrong thing here....

This patch is marked 'fixed' but the current CVS version of the module (HEAD) still contains the bug. I am assuming that the status was wrongly set to 'fixed' when the patch was submitted (as 'fixed' appears to be the default state for new issues).

I've set the status to 'patch', and given that this breaks the main navigation system and interferes with the creation of custom menus, I'm marking it as 'critical'.

I hope this is all okay.

killes@www.drop.org’s picture

killes@www.drop.org commented

Committed to head and 4.5, thanks.

jsloan’s picture

jsloan commented
Priority: Critical » Normal

Sorry for the confusion... I didn't even notice that the default status was "fixed" - thanks for cleaning up after me!

Anonymous’s picture

(not verified) commented