Sitemenu: display only terms that do have nodes associated with them

Posted by GiorgosK on February 27, 2007 at 12:42am

Issue Summary

I tweaked the code to do just that if anybody is interested

//line 268
      if ($get_node_count && $mode == 'page') {

//becomes
      if ($get_node_count ) {



//and lines 439-444

    if ($term->depth < $tree[$m+1]->depth) {
      $output .= "<li class=\"expanded\">". $link ."\n";
    }
    else {
      $output .= "<li class=\"leaf\">". $link ."</li>\n";  
    }

//become
if ($term->count) {//addition

    if ($term->depth < $tree[$m+1]->depth) {
      $output .= "<li class=\"expanded\">". $link ."\n";
    }
    else {
	  $output .= "<li class=\"leaf\">". $link ."</li>\n";  
    }
}//addition

or it could also be incorporated in the code with a configuration setting of whether to display terms with no nodes
(but I am not familiar with the drupal code yet to do it)

Comments

#1

Posted by theunsd on March 25, 2007 at 12:15pm

Thanks, this really helped me a lot. It would be nice if the developer built that in as a configuration setting. I'm sure we aren't the only 2 guys who have needed this. Thanks again!

#2

Posted by kbahey on March 25, 2007 at 4:36pm

Status:

active

» needs work

If someone prepares a proper tested working patch, with an optional settings, I will commit.

#3

Posted by systemparadox on November 3, 2007 at 1:02am

Status:

needs work

» reviewed & tested by the community

Patch attached. Unlike the modification above it still displays the nodes. Adds an option to the settings page.
Empty vocabularies will still be shown, but I don't think this matters. You could always disable them manually in the config page.