diff --git a/includes/menu.inc b/includes/menu.inc index b5a64af254f..367369dcba3 100644 --- a/includes/menu.inc +++ b/includes/menu.inc @@ -1787,8 +1787,11 @@ function _menu_navigation_links_rebuild($menu) { $existing_item = db_fetch_array(db_query("SELECT mlid, menu_name, plid, customized, has_children, updated FROM {menu_links} WHERE link_path = '%s' AND module = '%s'", $item['link_path'], 'system')); if ($existing_item) { $item['mlid'] = $existing_item['mlid']; - $item['menu_name'] = $existing_item['menu_name']; - $item['plid'] = $existing_item['plid']; + // A change in hook_menu may move the link to a different menu + if (empty($item['menu_name']) || ($item['menu_name'] == $existing_item['menu_name'])) { + $item['menu_name'] = $existing_item['menu_name']; + $item['plid'] = $existing_item['plid']; + } $item['has_children'] = $existing_item['has_children']; $item['updated'] = $existing_item['updated']; } diff --git a/modules/simpletest/tests/hook_menu.info b/modules/simpletest/tests/hook_menu.info new file mode 100644 index 00000000000..facbf6882f3 --- /dev/null +++ b/modules/simpletest/tests/hook_menu.info @@ -0,0 +1,8 @@ +; $Id$ +name = "Hook menu tests" +description = "Support module for menu hook testing." +package = Testing +version = VERSION +core = 7.x +files[] = hook_menu.module +hidden = TRUE diff --git a/modules/simpletest/tests/hook_menu.module b/modules/simpletest/tests/hook_menu.module new file mode 100644 index 00000000000..3b11fa042c0 --- /dev/null +++ b/modules/simpletest/tests/hook_menu.module @@ -0,0 +1,20 @@ + t('Test menu_name router item'), + 'page callback' => 'node_save', + 'menu_name' => isset($_GET["hook_menu_name"]) ? $_GET["hook_menu_name"] : 'original', + ); + return $items; +} \ No newline at end of file diff --git a/modules/simpletest/tests/menu.test b/modules/simpletest/tests/menu.test new file mode 100644 index 00000000000..93e4a4a5328 --- /dev/null +++ b/modules/simpletest/tests/menu.test @@ -0,0 +1,47 @@ + t('Hook menu tests'), + 'description' => t('Test menu hook functionality.'), + 'group' => t('Menu'), + ); + } + + /** + * Implementation of setUp(). + */ + function setUp() { + // Enable dummy module that implements hook_menu. + parent::setUp('hook_menu'); + } + + /** + * Tests for menu_name parameter for hook_menu(). + */ + function testMenuName() { + $admin_user = $this->drupalCreateUser(array('administer site configuration')); + $this->drupalLogin($admin_user); + + $sql = "SELECT menu_name FROM {menu_links} WHERE router_path = 'menu_name_test'"; + $name = db_result(db_query($sql)); + $this->assertEqual($name, 'original', t('Menu name is "original".')); + + // Force a menu rebuild by going to the modules page. + $this->drupalGet('admin/build/modules', array('query' => array("hook_menu_name" => 'changed'))); + + $sql = "SELECT menu_name FROM {menu_links} WHERE router_path = 'menu_name_test'"; + $name = db_result(db_query($sql)); + $this->assertEqual($name, 'changed', t('Menu name was successfully changed after rebuild.')); + } +}