' . t('This page shows you all available administration tasks for each module.') . '
'; case 'admin/appearance': $output = '' . t('Set and configure the default theme for your website. Alternative themes are available.', array('@themes' => 'http://drupal.org/project/themes')) . '
'; @@ -521,9 +524,9 @@ 'file' => 'system.admin.inc', ); $items['admin'] = array( - 'title' => 'Administer', + 'title' => 'Administration', 'access arguments' => array('access administration pages'), - 'page callback' => 'system_main_admin_page', + 'page callback' => 'system_admin_menu_block_page', 'weight' => 9, 'menu_name' => 'management', 'theme callback' => 'variable_get', @@ -538,15 +541,12 @@ 'file' => 'system.admin.inc', ); $items['admin/by-task'] = array( - 'title' => 'By task', - 'page callback' => 'system_main_admin_page', - 'access arguments' => array('access administration pages'), + 'title' => 'Administration', 'type' => MENU_DEFAULT_LOCAL_TASK, - 'file' => 'system.admin.inc', 'weight' => -20, ); - $items['admin/by-module'] = array( - 'title' => 'By module', + $items['admin/index'] = array( + 'title' => 'Index', 'page callback' => 'system_admin_by_module', 'access arguments' => array('access administration pages'), 'type' => MENU_LOCAL_TASK, @@ -2008,6 +2008,15 @@ */ function system_admin_menu_block($item) { $cache = &drupal_static(__FUNCTION__, array()); + // If we are calling this function for a menu item that corresponds to a + // local task (for example, admin/by-task), then we want to retrieve the + // parent item's child links, not this item's (since this item won't have + // any). + // @todo Not sure if this is completely correct. + if ($item['tab_root'] != $item['path']) { + $item = menu_get_item($item['tab_root']); + } + if (!isset($item['mlid'])) { $item += db_query("SELECT mlid, menu_name FROM {menu_links} ml WHERE ml.router_path = :path AND module = 'system'", array(':path' => $item['path']))->fetchAssoc(); } @@ -2017,26 +2026,12 @@ } $content = array(); - $default_task = NULL; - $has_subitems = FALSE; $result = db_query(" - SELECT m.load_functions, m.to_arg_functions, m.access_callback, m.access_arguments, m.page_callback, m.page_arguments, m.delivery_callback, m.title, m.title_callback, m.title_arguments, m.theme_callback, m.theme_arguments, m.type, m.description, m.path, m.weight as router_weight, ml.* + SELECT m.load_functions, m.to_arg_functions, m.access_callback, m.access_arguments, m.page_callback, m.page_arguments, m.delivery_callback, m.title, m.title_callback, m.title_arguments, m.theme_callback, m.theme_arguments, m.type, m.description, ml.* FROM {menu_router} m LEFT JOIN {menu_links} ml ON m.path = ml.router_path - WHERE (ml.plid = :plid AND ml.menu_name = :name AND hidden = 0) OR (m.tab_parent = :path AND m.type IN (:local_task, :default_task))", array(':plid' => $item['mlid'], ':name' => $item['menu_name'], ':path' => $item['path'], ':local_task' => MENU_LOCAL_TASK, ':default_task' => MENU_DEFAULT_LOCAL_TASK), array('fetch' => PDO::FETCH_ASSOC)); + WHERE ml.plid = :plid AND ml.menu_name = :name AND hidden = 0", array(':plid' => $item['mlid'], ':name' => $item['menu_name']), array('fetch' => PDO::FETCH_ASSOC)); foreach ($result as $link) { - if (!isset($link['link_path'])) { - // If this was not an actual link, fake the tab as a menu link, so we - // don't need to special case it beyond this point. - $link['link_title'] = $link['title']; - $link['link_path'] = $link['path']; - $link['options'] = 'a:0:{}'; - $link['weight'] = $link['router_weight']; - } - else { - // We found a non-tab subitem, remember that. - $has_subitems = TRUE; - } _menu_link_translate($link); if (!$link['access']) { continue; @@ -2055,14 +2050,6 @@ // The weight is offset so it is always positive, with a uniform 5-digits. $key = (50000 + $link['weight']) . ' ' . drupal_strtolower($link['title']) . ' ' . $link['mlid']; $content[$key] = $link; - if ($link['type'] == MENU_DEFAULT_LOCAL_TASK) { - $default_task = $key; - } - } - if ($has_subitems) { - // If we've had at least one non-tab subitem, remove the link for the - // default task, since that is already broken down to subitems. - unset($content[$default_task]); } ksort($content); $cache[$item['mlid']] = $content; @@ -2761,16 +2748,19 @@ * * @param $module * Module name. + * @param $info + * Module's info array from {system} table (as built by system_get_info). + * * @return * An array of task links. */ -function system_get_module_admin_tasks($module) { +function system_get_module_admin_tasks($module, $info) { $items = &drupal_static(__FUNCTION__, array()); if (empty($items)) { $result = db_query(" SELECT m.load_functions, m.to_arg_functions, m.access_callback, m.access_arguments, m.page_callback, m.page_arguments, m.delivery_callback, m.title, m.title_callback, m.title_arguments, m.theme_callback, m.theme_arguments, m.type, ml.* - FROM {menu_links} ml INNER JOIN {menu_router} m ON ml.router_path = m.path WHERE ml.link_path LIKE 'admin/%' AND hidden >= 0 AND module = 'system' AND m.number_parts > 2", array(), array('fetch' => PDO::FETCH_ASSOC)); + FROM {menu_links} ml INNER JOIN {menu_router} m ON ml.router_path = m.path WHERE ml.link_path LIKE 'admin/%' AND hidden >= 0 AND module = 'system' AND m.number_parts > 1 AND page_callback <> 'system_admin_menu_block_page'", array(), array('fetch' => PDO::FETCH_ASSOC)); foreach ($result as $item) { _menu_link_translate($item); if ($item['access']) { @@ -2783,15 +2773,28 @@ $admin_tasks = array(); $admin_task_count = 0; // Check for permissions. + $permissions_menu_item = menu_get_item('admin/people/permissions'); + $permissions_menu_item['description'] = NULL; + $permissions_menu_item['title'] = t('Configure @module permissions', array('@module' => $info['name'])); if (in_array($module, module_implements('permission')) && $admin_access) { - $admin_tasks[-1] = l(t('Configure permissions'), 'admin/people/permissions', array('fragment' => 'module-' . $module)); + $permissions_menu_item['link_path'] = 'admin/people/permissions'; + $permissions_menu_item['localized_options']['fragment'] = 'module-' . $module; + $admin_tasks[-1] = $permissions_menu_item; } // Check for menu items that are admin links. if (in_array($module, module_implements('menu')) && $menu = module_invoke($module, 'menu')) { foreach (array_keys($menu) as $path) { if (isset($items[$path])) { - $admin_tasks[$items[$path]['title'] . $admin_task_count ++] = l($items[$path]['title'], $path); + // The link 'description' ? either derived from the hook_menu + // 'description' or entered by the user via menu module ? is saved as + // the title attribute. The title attribute is then unset to reduce + // redundancy in admin pages for screen readers. + if (!empty($items[$path]['localized_options']['attributes']['title'])) { + $items[$path]['description'] = $items[$path]['localized_options']['attributes']['title']; + unset($items[$path]['localized_options']['attributes']['title']); + } + $admin_tasks[$items[$path]['title'] . $admin_task_count ++] = $items[$path]; } } }