Blank browser screen when going to Menu Trails admin page

There are a couple of bugs caused by Menutrails unfortunately assuming the presence of the Taxonomy module, so when it's not things go bang. The admin screen blows up on 252 for a similar reason.

I've made a fix by:

• Wrapping the foreach block at line 130 with an if statement:
  
  <?php
  if ($node->taxonomy) {
    foreach ($node->taxonomy as $term) {
      if ($term_trails[$term->tid]) {
        $href = $term_trails[$term->tid];
      }
    }
  }
?>
• Wrapping the request to get taxonomy vocabularies on line 252 with a module_exists check:
  
  <?php
  if (module_exists("taxonomy")) {
    $vocabs = taxonomy_get_vocabularies();
  }
?>
• Similar on line 270 now that $vocabs can be empty:
  
  <?php
  if ($vocabs) {
    foreach ($vocabs as $vocab) {
      if ($vocab->tags != 1) {
        // tagging gets out of hand too fast, so we disallow
        $form[$vocab->vid]['menutrails_terms'] = array(
          '#tree' => TRUE,
          '#type' => 'fieldset',
          '#collapsible' => TRUE,
          '#collapsed' => TRUE,
          '#title' => t('Categories:') ." $vocab->name",
        );
        $terms = taxonomy_get_tree($vocab->vid);
        foreach ($terms as $term) {
           $form[$vocab->vid]['menutrails_terms'][$term->tid] = array('#type' => 'select',
            '#title' => t('Parent item for') ." $term->name",
            '#default_value' => $term_trails[$term->tid],
            '#options' => $options,
          );
        }
      }
    }
  }
?>

After the above 3 checks are added, there are no more problems for me.

Jim

Ahhh bugger. I spotted the above is already reported (and fixed in dev) in #313556: Missing dependency declaration for taxonomy module just after I posted the above.

Sorry for the delayed response. I implemented the dev version and now it works fine. Thanks for saving my day and thanks to Josh K for fixing the problem!!

subscribing