Index: includes/menu.inc =================================================================== RCS file: /cvs/drupal/drupal/includes/menu.inc,v retrieving revision 1.427 diff -u -r1.427 menu.inc --- includes/menu.inc 2 Dec 2010 20:18:23 -0000 1.427 +++ includes/menu.inc 7 Dec 2010 16:51:07 -0000 @@ -49,8 +49,9 @@ * Access to the callback functions is also protected by the menu system. * The "access callback" with an optional "access arguments" of each menu * item is called before the page callback proceeds. If this returns TRUE, - * then access is granted; if FALSE, then access is denied. Menu items may - * omit this attribute to use the value provided by an ancestor item. + * then access is granted; if FALSE, then access is denied. Default local task + * menu items (see next paragraph) may omit this attribute to use the value + * provided by the parent item. * * In the default Drupal interface, you will notice many links rendered as * tabs. These are known in the menu system as "local tasks", and they are Index: modules/system/system.api.php =================================================================== RCS file: /cvs/drupal/drupal/modules/system/system.api.php,v retrieving revision 1.218 diff -u -r1.218 system.api.php --- modules/system/system.api.php 1 Dec 2010 00:23:36 -0000 1.218 +++ modules/system/system.api.php 7 Dec 2010 16:51:07 -0000 @@ -1002,21 +1002,22 @@ * $items['admin/config/foo'] = array( * 'title' => 'Foo settings', * 'type' => MENU_NORMAL_ITEM, - * // page callback, etc. need to be added here + * // Page callback, etc. need to be added here. * ); * // Make "Global settings" the main tab on the "Foo settings" page * $items['admin/config/foo/global'] = array( * 'title' => 'Global settings', * 'type' => MENU_DEFAULT_LOCAL_TASK, - * // access callback, page callback, and theme callback will be inherited - * // from 'admin/config/foo', if not specified here to override + * // Access callback, page callback, and theme callback will be inherited + * // from 'admin/config/foo', if not specified here to override. * ); * // Make an additional tab called "Node settings" on "Foo settings" * $items['admin/config/foo/node'] = array( * 'title' => 'Node settings', * 'type' => MENU_LOCAL_TASK, - * // access callback, page callback, and theme callback will be inherited - * // from 'admin/config/foo', if not specified here to override + * // Page callback and theme callback will be inherited from + * // 'admin/config/foo', if not specified here to override. + * // Need to add access callback or access arguments. * ); * @endcode * @@ -1045,9 +1046,14 @@ * rights to this menu item, and FALSE if not. It can also be a boolean * constant instead of a function, and you can also use numeric values * (will be cast to boolean). Defaults to user_access() unless a value is - * inherited from a parent menu item. + * inherited from the parent menu item; only MENU_DEFAULT_LOCAL_TASK items + * can inherit access callbacks. To use the user_access() default callback, + * you must specify the permission to check as 'access arguments' (see + * below). * - "access arguments": An array of arguments to pass to the access callback - * function, with path component substitution as described above. + * function, with path component substitution as described above. If the + * access callback is inherited (see above), the access arguments will be + * inherited with it, unless overridden in the child menu item. * - "theme callback": (optional) A function returning the machine-readable * name of the theme that will be used to render the page. If not provided, * the value will be inherited from a parent menu item. If there is no