diff --git a/core/includes/form.inc b/core/includes/form.inc index ed6929c..234e640 100644 --- a/core/includes/form.inc +++ b/core/includes/form.inc @@ -1527,6 +1527,9 @@ function form_process_container($element, &$form_state) { */ function theme_tableselect($variables) { $element = $variables['element']; + $table = array( + '#theme' => 'table', + ); $rows = array(); $header = $element['#header']; if (!empty($element['#options'])) { @@ -1566,7 +1569,7 @@ function theme_tableselect($variables) { // checkboxes/radios in the first table column. if ($element['#js_select']) { // Add a "Select all" checkbox. - drupal_add_library('system', 'drupal.tableselect'); + $table['#attached']['library'][] = array('system', 'drupal.tableselect'); array_unshift($header, array('class' => array('select-all'))); } else { @@ -1575,8 +1578,7 @@ function theme_tableselect($variables) { array_unshift($header, ''); } } - $table = array( - '#theme' => 'table', + $table += array( '#header' => $header, '#rows' => $rows, '#empty' => $element['#empty'], @@ -2778,31 +2780,23 @@ function theme_form_element($variables) { $prefix = isset($element['#field_prefix']) ? '' . $element['#field_prefix'] . ' ' : ''; $suffix = isset($element['#field_suffix']) ? ' ' . $element['#field_suffix'] . '' : ''; + $form_element_label = array( + '#theme' => 'form_element_label', + '#required' => !empty($element['#required']) ? $element['#required'] : '', + '#title' => !empty($element['#title']) ? $element['#title'] : '', + '#title_display' => !empty($element['#title_display']) ? $element['#title_display'] : '', + '#id' => !empty($element['#id']) ? $element['#id'] : '', + ); + switch ($element['#title_display']) { case 'before': case 'invisible': - $form_element_label = array( - '#theme' => 'form_element_label', - '#required' => !empty($variables['#required']) ? $variables['#required'] : '', - '#title' => !empty($variables['#title']) ? $variables['#title'] : '', - '#id' => !empty($variables['#id']) ? $variables['#id'] : '', - '#value' => !empty($variables['#value']) ? $variables['#value'] : '', - '#description' => !empty($variables['#description']) ? $variables['#description'] : '', - ); $output .= ' ' . drupal_render($form_element_label); $output .= ' ' . $prefix . $element['#children'] . $suffix . "\n"; break; case 'after': $output .= ' ' . $prefix . $element['#children'] . $suffix; - $form_element_label = array( - '#theme' => 'form_element_label', - '#required' => !empty($variables['#required']) ? $variables['#required'] : '', - '#title' => !empty($variables['#title']) ? $variables['#title'] : '', - '#id' => !empty($variables['#id']) ? $variables['#id'] : '', - '#value' => !empty($variables['#value']) ? $variables['#value'] : '', - '#description' => !empty($variables['#description']) ? $variables['#description'] : '', - ); $output .= ' ' . drupal_render($form_element_label) . "\n"; break; @@ -2875,10 +2869,7 @@ function theme_form_element_label($variables) { } // If the element is required, a required marker is appended to the label. - $form_required_marker = array( - '#theme' => 'form_required_marker', - '#element' => $element, - ); + $form_required_marker = array('#theme' => 'form_required_marker'); $required = !empty($element['#required']) ? drupal_render($form_required_marker) : ''; $title = filter_xss_admin($element['#title']);