The 2026 Drupal Business Survey is open. Your response will help shape the future of the Drupal business ecosystem.http://api.drupal.org/api/function/hook_menu/6
There might be other stuff too. This one just happened to bite me today.
| Comment | File | Size | Author |
|---|---|---|---|
| #9 | hook_menu_docs.patch | 978 bytes | matason |
Comments
Comment #1
Barry Madore commentedMoving
Comment #2
lilou commentedFor
hook_menu()@file tag, see : http://api.drupal.org/api/file/developer/hooks/core.php/7/sourceComment #3
Anonymous (not verified) commentedAutomatically closed -- issue fixed for two weeks with no activity.
Comment #4
webchickNope, sorry, what I mean is that in addition to other properties listed there like 'page callback' and such, there is also a 'file' property which needs documenting.
There is also a 'file path' property.
Comment #5
matason commented“file”: Optional. The name of the file containing the function referred to in either “page callback” or “page arguments”. If this file does not reside in your module directory you'll need to specify the path using “file path”.
“file path”: Optional. The path to the file referred to in “file”.
How's that?
Comment #6
alex ua commented@matason- You didn't actually attach a patch, so this should really remain marked "active".
Comment #7
matason commentedOkay, well thanks...
I'll apply for a CVS account and then I'll be able to submit a patch (unless I'm missing something but I can't seem to checkout contributions/docs/developer anonymously)
If in the meantime someone with CVS could submit a patch that'll be great.
Comment #8
alex ua commented@matason - the api docs are being automatically generated (by and large) by comments within the modules themselves. Thus, you don't need access to "contributions/docs/developer" to make a patch against menu.api.php, you just need to make a patch against the head version of modules/menu/menu.api.php
http://api.drupal.org/api/file/modules/menu/menu.api.php/7/source
Comment #9
matason commentedHey thanks Alex UA, I've attached a patch, hopefully it's useful!
Cheers,
Chris
Comment #10
catchThere's no file in hook_menu() in Drupal 7 due to the registry unless I'm very much mistaken, so bumping down to Drupal 6 and the documentation queue.
Comment #11
add1sun commentedEr, any docs on api.drupal.org belong in the core queue where people with CVS accounts will actually take care of things. Moving it back. :p
Comment #12
webchickOops! catch is correct. These properties were removed in D7 in favour of the registry, so this only needs to be committed to the 6.x branch of contributions/docs/developer/hooks/core.php.
@matason: Yay! :D Thanks a lot! Is there any chance you'd want to expand your patching prowess to patch against that file rather than Drupal core? (Sorry about the confusion! :\)
Comment #13
matason commentedHello webchick,
Yes certainly, I will apply for a CVS account this evening - I'd love to "expand my patching prowess" ;)
Best,
Chris
Comment #14
matason commentedI'm still waiting for a decision on my CVS application, will hopefully be able to dive in and make the patch soon!
Comment #15
matason commented"There might be other stuff too."
There is, "title callback" along with "title arguments", adding as a reminder to document them also.
Comment #16
add1sun commented@matason, just to be clear you don't need a CVS account to create a patch.
Comment #18
jhodgdonThis issue appears to have been fixed by someone else wheile working on another patch, I would guess. In any case, the file and file path components of hook_menu() are currently documented on api.drupal.org.