hook_menu example should be changed to not suggest admin/config/foo as path

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.

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.

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

Subscribe

Exemple works for me with the patch

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

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

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

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

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

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