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:

    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:
  

    if (module_exists("taxonomy")) {
      $vocabs = taxonomy_get_vocabularies();
    }
  

• Similar on line 270 now that $vocabs can be empty:
  

    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.

subscribing