If we have a term with the counter to 0 and that have terms children, the tree is broken and does not correspond with reality.

Example real taxonomy tree:

+ Parent Term 1 (>0)
+ Parent Term 2 (0)
|--- SubTerm 2.1 (>0)
|--- SubTerm 2.2 (>0)

Result with site map:

+ Parent Term 1 (>0)
|--- SubTerm 2.1 (>0)
|--- SubTerm 2.2 (>0)
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

darrell_ulm’s picture

Assigned: Unassigned » darrell_ulm
Category: bug » task
Status: Active » Needs work

Yep, I agree, this does not work for taxonomy trees. The taxonomy tag depth is not correct.

Looking into it, and accepting patches of course.

darrell_ulm’s picture

Status: Needs work » Active

Changed to active until there is a patch.

darrell_ulm’s picture

Status: Active » Closed (duplicate)

This is a duplicate of http://drupal.org/node/1593556 and referring to the new one because it is part of a larger issue of the category depth not working in more than just this case.

tomyinhauser’s picture

Issue summary: View changes
Status: Closed (duplicate) » Active

The related issue does NOT solve this problem.

tomyinhauser’s picture

The attached patch considers the content count in the whole hierarchy to filter the taxonomy terms.

Status: Needs review » Needs work

The last submitted patch, 5: site_map-taxonomy-hierarchy-count-1348022-5-D7.patch, failed testing.

The last submitted patch, 5: site_map-taxonomy-hierarchy-count-1348022-5-D7.patch, failed testing.

tomyinhauser’s picture

Status: Needs work » Needs review

Status: Needs review » Needs work