Sitemenu: display only terms that do have nodes associated with them [#122989]

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)

Comment	File	Size	Author
#3	sitemenu-hide_empty.patch	1.92 KB	systemparadox

Comments

Comment #1

theunsd commented 25 March 2007 at 12:15

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!

Comment #2

kbahey commented 25 March 2007 at 16:36

Status:

Active

» Needs work

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

Comment #3

systemparadox commented 3 November 2007 at 01:02

Status:

Needs work

» Reviewed & tested by the community

Status	File	Size
new	sitemenu-hide_empty.patch	1.92 KB

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.