Thnx for the theme. Makes using bootstrap fairly easy. Yay!

Minor problem in form.inc, bootstrap_form_alter(). Code assumes $form['actions']['#type'] is set - might not be.

Generates error: Notice: Undefined index: #type in bootstrap_form_alter() (line 21 of D:\xampp\htdocs\km\sites\all\themes\bootstrap\includes\form.inc).

Current code:

  // Only wrap in container for certain form
  if (isset($form['#form_id']) && !in_array($form['#form_id'], $form_ids) 
      && !isset($form['#node_edit_form']) 
      && isset($form['actions'])
      && ($form['actions']['#type'] == 'actions')) { // <- $form['actions']['#type']  might not exist
    $form['actions']['#theme_wrappers'] = array();
  }

Add a line:

  // Only wrap in container for certain form
  if (isset($form['#form_id']) && !in_array($form['#form_id'], $form_ids) 
      && !isset($form['#node_edit_form']) 
      && isset($form['actions'])
      && isset( $form['actions']['#type'] ) // <-new line
      && ($form['actions']['#type'] == 'actions')) {
    $form['actions']['#theme_wrappers'] = array();
  }

Kieran

Comments

markhalliwell’s picture

Issue summary: View changes
Status: Active » Closed (won't fix)

This issue has been closed while cleaning up the issue queue. This has likely already been fixed in 7.x-3.x. If it has not, please create a new issue describing the exact issue with that version. The 7.x-2.x branch currently only receives security fixes.