Index: includes/menu.inc =================================================================== RCS file: /cvs/drupal/drupal/includes/menu.inc,v retrieving revision 1.255.2.28 diff -u -p -r1.255.2.28 menu.inc --- includes/menu.inc 9 Feb 2009 16:28:21 -0000 1.255.2.28 +++ includes/menu.inc 10 Feb 2009 01:18:40 -0000 @@ -1668,30 +1668,36 @@ function menu_rebuild() { /** * Collect, alter and store the menu definitions. */ -function menu_router_build($reset = FALSE) { - static $menu; +function menu_router_build() { - if (!isset($menu) || $reset) { - if (!$reset && ($cache = cache_get('router:', 'cache_menu')) && isset($cache->data)) { - $menu = $cache->data; - } - else { - // We need to manually call each module so that we can know which module - // a given item came from. - $callbacks = array(); - foreach (module_implements('menu') as $module) { - $router_items = call_user_func($module .'_menu'); - if (isset($router_items) && is_array($router_items)) { - foreach (array_keys($router_items) as $path) { - $router_items[$path]['module'] = $module; - } - $callbacks = array_merge($callbacks, $router_items); + // We need to manually call each module so that we can know which module + // a given item came from. + $callbacks = array(); + foreach (module_implements('menu') as $module) { + $router_items = call_user_func($module .'_menu'); + if (isset($router_items) && is_array($router_items)) { + foreach (array_keys($router_items) as $path) { + $router_items[$path]['module'] = $module; } + $callbacks = array_merge($callbacks, $router_items); } - // Alter the menu as defined in modules, keys are like user/%user. - drupal_alter('menu', $callbacks); - $menu = _menu_router_build($callbacks); } + // Alter the menu as defined in modules, keys are like user/%user. + drupal_alter('menu', $callbacks); + $menu = _menu_router_build($callbacks); + _menu_router_store($menu); + + return $menu; +} + +/** + * Helper function to store the menu router if we have it in memory.. + */ +function _menu_router_store($new_menu = NULL) { + static $menu = NULL; + + if (isset($new_menu)) { + $menu = $new_menu; } return $menu; } @@ -1841,9 +1847,8 @@ function _menu_delete_item($item, $force * saved. */ function menu_link_save(&$item) { - $menu = menu_router_build(); - drupal_alter('menu_link', $item, $menu); + drupal_alter('menu_link', $item); // This is the easiest way to handle the unique internal path '', // since a path marked as external does not need to match a router path. @@ -1960,7 +1965,7 @@ function menu_link_save(&$item) { else { // Find the router path which will serve this path. $item['parts'] = explode('/', $item['link_path'], MENU_MAX_PARTS); - $item['router_path'] = _menu_find_router_path($menu, $item['link_path']); + $item['router_path'] = _menu_find_router_path($item['link_path']); } } db_query("UPDATE {menu_links} SET menu_name = '%s', plid = %d, link_path = '%s', @@ -2021,19 +2026,24 @@ function _menu_set_expanded_menus() { /** * Find the router path which will serve this path. * - * @param $menu - * The full built menu. * @param $link_path * The path for we are looking up its router path. * @return * A path from $menu keys or empty if $link_path points to a nonexisting * place. */ -function _menu_find_router_path($menu, $link_path) { - $parts = explode('/', $link_path, MENU_MAX_PARTS); +function _menu_find_router_path($link_path) { + $menu = _menu_router_store(); + $router_path = $link_path; - if (!isset($menu[$router_path])) { - list($ancestors) = menu_get_ancestors($parts); + $parts = explode('/', $link_path, MENU_MAX_PARTS); + list($ancestors, $placeholders) = menu_get_ancestors($parts); + + if (!isset($menu)) { + // Not during a menu rebuild, so look up in the database. + $router_path = (string)db_result(db_query_range('SELECT path FROM {menu_router} WHERE path IN ('. implode (',', $placeholders) .') ORDER BY fit DESC', $ancestors, 0, 1)); + } + elseif (!isset($menu[$router_path])) { $ancestors[] = ''; foreach ($ancestors as $key => $router_path) { if (isset($menu[$router_path])) { @@ -2391,7 +2401,7 @@ function _menu_router_build($callbacks) $masks = array_keys($masks); rsort($masks); variable_set('menu_masks', $masks); - cache_set('router:', $menu, 'cache_menu'); + return $menu; } Index: modules/menu/menu.module =================================================================== RCS file: /cvs/drupal/drupal/modules/menu/menu.module,v retrieving revision 1.157.2.3 diff -u -p -r1.157.2.3 menu.module --- modules/menu/menu.module 10 May 2008 06:53:53 -0000 1.157.2.3 +++ modules/menu/menu.module 10 Feb 2009 01:18:40 -0000 @@ -254,8 +254,7 @@ function _menu_parents_recurse($tree, $m * Reset a system-defined menu item. */ function menu_reset_item($item) { - $router = menu_router_build(); - $new_item = _menu_link_build($router[$item['router_path']]); + $new_item = _menu_link_build(menu_get_item($item['router_path'])); foreach (array('mlid', 'has_children') as $key) { $new_item[$key] = $item[$key]; } Index: modules/system/system.install =================================================================== RCS file: /cvs/drupal/drupal/modules/system/system.install,v retrieving revision 1.238.2.8 diff -u -p -r1.238.2.8 system.install --- modules/system/system.install 14 Jan 2009 21:36:16 -0000 1.238.2.8 +++ modules/system/system.install 10 Feb 2009 01:18:40 -0000 @@ -2532,6 +2532,13 @@ function system_update_6048() { return $ret; } +/** + * Clear any router blobs stored in the cache table. + */ +function system_update_6049() { + cache_clear_all('router:', 'cache_menu', TRUE); + return array(); +} /** * @} End of "defgroup updates-5.x-to-6.x"