The hook_menu() documentation describes a settings page at admin/config/foo as an example for how to implement tabs. However, if you follow this example, your page won't show up, because pages directly under admin/config don't appear on admin/config. Per https://drupal.org/node/549094#configuration, you need to put your pages under a sub-category, e.g. admin/config/development/foo.

CommentFileSizeAuthor
#10 hook_menu_example-1257638-9.patch1.71 KBbès
#7 hook_menu_example-1257638-7.patch1.71 KBbès
#3 hook_menu_example-1257638-3.patch1.97 KBmichaellenahan

Comments

dave reid’s picture

dave reid
he/him
Primary language English
Location Nebraska USA
commented

We need to change this example to not live under admin/config as it gives a false impression that it should be used for creating new config menu categories when it should not.

jhodgdon’s picture

jhodgdon
she/her
Primary language English
commented
Title: Documentation problem with hook_menu » hook_menu example should be changed to not suggest admin/config/foo as path
Version: 7.x-dev » 8.x-dev
Issue tags: +Needs backport to D7

How about making a sub-page of one of the existing admin categories? Anyway, thanks for reporting and it does sound like something that should be fixed.

michaellenahan’s picture

michaellenahan commented
Status: Active » Needs review
StatusFileSize
new hook_menu_example-1257638-3.patch1.97 KB

Hello world, here we are at DrupalCon London 2011
LinL and michaellenahan

linl’s picture

linl commented

Subscribe

bès’s picture

bès commented

Exemple works for me with the patch

jhodgdon’s picture

jhodgdon
she/her
Primary language English
commented
Status: Needs review » Needs work

Thanks!

The patch looks good -- I think the new example is fine. However, I think we need to remove this bit from the end of the patch file:

diff --git a/scripts/drupal.sh b/scripts/drupal.sh
old mode 100755
new mode 100644
diff --git a/scripts/password-hash.sh b/scripts/password-hash.sh
old mode 100755
new mode 100644
diff --git a/scripts/run-tests.sh b/scripts/run-tests.sh
old mode 100755
new mode 100644
bès’s picture

bès commented
Status: Needs work » Needs review
StatusFileSize
new hook_menu_example-1257638-7.patch1.71 KB

Here is a new version without the strange things at the end

bès’s picture

bès commented

Here is a new version without the strange things at the end

Status: Needs review » Needs work

The last submitted patch, hook_menu_example-1257638-7.patch, failed testing.

bès’s picture

bès commented
StatusFileSize
new hook_menu_example-1257638-9.patch1.71 KB

Whut ? no patch, double post ? hum OK let's do it again

bès’s picture

bès commented
Status: Needs work » Needs review
jhodgdon’s picture

jhodgdon
she/her
Primary language English
commented
Status: Needs review » Reviewed & tested by the community

Looks good, thanks! This should go into d7/d8 assuming it applies to 7.

dries’s picture

dries commented
Status: Reviewed & tested by the community » Fixed

Committed to 7.x and 8.x. Thanks.

Automatically closed -- issue fixed for 2 weeks with no activity.