Currently, there is this code in hook_menu():

  $items['ahah_example/simplest_ahah/callback'] = array(
    'page callback' => 'ahah_example_simplest_callback',
    'access callback' => TRUE,
    'file' => 'simplest_ahah.inc',
   );

Silly menu item

Because type is not specified, this ends up showing as a blank entry in the navigation menu on the menu administration page.

This should be changed to type => MENU_CALLBACK.

CommentFileSizeAuthor
#5 examples-ahah-fix-856656-5.patch845 byteswebchick
#2 examples-ahah-fix-856656-2.patch723 byteswebchick

Comments

webchick’s picture

webchick
she/they
Primary language English
Location Vancouver 🇨🇦
commented
Assigned: Unassigned » webchick
webchick’s picture

webchick
she/they
Primary language English
Location Vancouver 🇨🇦
commented
Status: Active » Needs review
StatusFileSize
new examples-ahah-fix-856656-2.patch723 bytes

Here's a fix.

rfay’s picture

rfay
he,him
Primary language English
Location Palisade, CO, USA
commented
Status: Needs review » Needs work 
+    'tyop' => MENU_CALLBACK,

Isn't this a big typo?

webchick’s picture

webchick
she/they
Primary language English
Location Vancouver 🇨🇦
commented 
+++ ahah_example.module	16 Jul 2010 20:05:01 -0000
@@ -28,7 +28,8 @@ function ahah_example_menu() {
+    'tyop' => MENU_CALLBACK,

Hey, I think you have a typo here.. that should actually "type".

Powered by Dreditor.

webchick’s picture

webchick
she/they
Primary language English
Location Vancouver 🇨🇦
commented
Status: Needs work » Needs review
StatusFileSize
new examples-ahah-fix-856656-5.patch845 bytes

Ok, I felt guilty... here's a re-roll of the patch.

rfay’s picture

rfay
he,him
Primary language English
Location Palisade, CO, USA
commented
Status: Needs review » Reviewed & tested by the community

Wow, thanks, that looks great. I'll commit when the bot comes back green.

THANKS!
-Randy

rfay’s picture

rfay
he,him
Primary language English
Location Palisade, CO, USA
commented
Status: Reviewed & tested by the community » Closed (duplicate)

Thanks!

Actually resolved in #857302: Add a simple validation example to AHAH example