This hook has three parameters; only one is documented.

CommentFileSizeAuthor
#4 788310.patch826 bytesjhodgdon
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

jhodgdon’s picture

Title: Documentation problem with hook_menu_local_tasks_alter » hook_menu_local_tasks_alter() doc is missing 2 parameters

Good catch!
http://api.drupal.org/api/function/hook_menu_local_tasks_alter/7

http://api.drupal.org/api/function/menu_local_tasks/7 is what calls drupal_alter() to invoke this hook, so you can check there to see what the parameters are, although since they're internally generated, there's no param doc to just copy in.

dawehner’s picture

Isn't this fixed now? I looked at the hook and every parameter is documented.

joachim’s picture

jhodgdon’s picture

Status: Active » Needs review
FileSize
826 bytes

Here's a patch.

As a note: the parameters were there in the hook function definition, but they didn't have @param documentation. This patch adds that documentation.

eustace’s picture

Successfully applied patch 788310.patch with no errors.

Patch adds documentation to $router_item and $root_path function parameters with proper '@param' prefixes.

Good Patch!

joachim’s picture

Status: Needs review » Reviewed & tested by the community

Thanks for the review eustace. Let's mark this RTBC accordingly :)

Dries’s picture

Status: Reviewed & tested by the community » Fixed

Committed to CVS HEAD. Thanks.

Status: Fixed » Closed (fixed)

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