It appears that docs weren't added to hook_menu() to describe the MENU_LOCAL_ACTION type constant.

The attached patch adds the description used in the constants API page: http://api.drupal.org/api/constant/MENU_LOCAL_ACTION/7

Comments

brianV’s picture

Improved the MENU_LOCAL_TASK description to hopefully make the difference between tasks and actions a little clearer.

The naming scheme is a WTF in it's own right, but I'll save that for #607656: rename one/both of MENU_LOCAL_ACTION / MENU_LOCAL_TASK.

jhodgdon’s picture

Status: Needs review » Needs work

Why take out the information about them being displayed as tabs?

brianV’s picture

Status: Needs work » Needs review
StatusFileSize
new1.26 KB

Added the info about TASKs being rendered as tabs, and tried to add similar about ACTIONS:

 *     - MENU_LOCAL_ACTION: Local actions are menu items that describe actions
 *       on the parent item such as adding a new user or block, and are
 *       rendered in the action-links list in your theme.
 *     - MENU_LOCAL_TASK: Local tasks are menu items that describe different
 *       displays of data, and are generally rendered as tabs.

I couldn't come up with better wording for Actions, since where and how the action-links list is rendered can and likely will vary much by theme.

catch’s picture

Priority: Critical » Normal
Status: Needs review » Reviewed & tested by the community

Looks fine.

dries’s picture

Status: Reviewed & tested by the community » Fixed

Committed to CVS HEAD. Thanks.

jhodgdon’s picture

Version: 7.x-dev » 6.x-dev
Status: Fixed » Patch (to be ported)

This probably needs a port to Drupal 6.

catch’s picture

Version: 6.x-dev » 7.x-dev
Status: Patch (to be ported) » Fixed

No MENU_LOCAL_ACTION in D6.

Status: Fixed » Closed (fixed)

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