The menu uses MENU_CALLBACK_ITEM but it's actually just MENU_CALLBACK.

CommentFileSizeAuthor
#1 942120_callback.patch871 bytesgreggles

Comments

greggles’s picture

greggles
he/him
Primary language English
Location Denver, Colorado, USA
commented
Status: Active » Needs review
StatusFileSize
new 942120_callback.patch871 bytes
greggles’s picture

greggles
he/him
Primary language English
Location Denver, Colorado, USA
commented
Title: MENU_CALLBACK » MENU_CALLBACK vs. MENU_CALLBACK_ITEM

better title.

And to be specific, the bug is that
1. Instead of using menu_callback the menu items are actually defined as menu_normal_item, which is the default for menu items.
2. This causes notices in sites that run with E_ALL.

markwk’s picture

markwk commented

This is fixed in D7 patch #942808: Drupal 7 port. Still looks like this issue remains in D6 version.

Drave Robber’s picture

Drave Robber commented
Status: Needs review » Reviewed & tested by the community

The patch in #1 fixes it for me.

Although this issue barely affects normal operation, these notices pop up on menu rebuild, for example, when drupal_flush_all_caches() is called in update_finished(); this can scare the stuff out of less experienced users, leading them to think update.php has failed.

oliverpolden’s picture

oliverpolden commented

This is still an issue. The latest dev version as of a few days ago still has MENU_CALLBACK_ITEM.