Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
This hook has three parameters; only one is documented.
Comment | File | Size | Author |
---|---|---|---|
#4 | 788310.patch | 826 bytes | jhodgdon |
Comments
Comment #1
jhodgdonGood 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.
Comment #2
dawehnerIsn't this fixed now? I looked at the hook and every parameter is documented.
Comment #3
joachim CreditAttribution: joachim commentedI only see one on http://api.drupal.org/api/function/hook_menu_local_tasks_alter/7
Comment #4
jhodgdonHere'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.
Comment #5
eustace CreditAttribution: eustace commentedSuccessfully applied patch 788310.patch with no errors.
Patch adds documentation to $router_item and $root_path function parameters with proper '@param' prefixes.
Good Patch!
Comment #6
joachim CreditAttribution: joachim commentedThanks for the review eustace. Let's mark this RTBC accordingly :)
Comment #7
Dries CreditAttribution: Dries commentedCommitted to CVS HEAD. Thanks.