- Patch #553106 by kiamlaluno, dereine, marcingy: _menu_overview_tree_form() access a not defined variable.

merge-requests/26/head
Dries Buytaert 2010-06-26 17:43:22 +00:00
parent 33e814ca77
commit 353ef4547a
1 changed files with 5 additions and 2 deletions

View File

@ -86,6 +86,9 @@ function menu_overview_form($form, &$form_state, $menu) {
/** /**
* Recursive helper function for menu_overview_form(). * Recursive helper function for menu_overview_form().
*
* @param $tree
* The menu_tree retrieved by menu_tree_data.
*/ */
function _menu_overview_tree_form($tree) { function _menu_overview_tree_form($tree) {
$form = &drupal_static(__FUNCTION__, array('#tree' => TRUE)); $form = &drupal_static(__FUNCTION__, array('#tree' => TRUE));
@ -105,7 +108,7 @@ function _menu_overview_tree_form($tree) {
$form[$mlid]['weight'] = array( $form[$mlid]['weight'] = array(
'#type' => 'weight', '#type' => 'weight',
'#delta' => 50, '#delta' => 50,
'#default_value' => isset($form_state[$mlid]['weight']) ? $form_state[$mlid]['weight'] : $item['weight'], '#default_value' => $item['weight'],
'#title_display' => 'invisible', '#title_display' => 'invisible',
'#title' => t('Weight for @row', array('@row' => $item['title'])), '#title' => t('Weight for @row', array('@row' => $item['title'])),
); );
@ -115,7 +118,7 @@ function _menu_overview_tree_form($tree) {
); );
$form[$mlid]['plid'] = array( $form[$mlid]['plid'] = array(
'#type' => 'hidden', '#type' => 'hidden',
'#default_value' => isset($form_state[$mlid]['plid']) ? $form_state[$mlid]['plid'] : $item['plid'], '#default_value' => $item['plid'],
); );
// Build a list of operations. // Build a list of operations.
$operations = array(); $operations = array();