Hi,
at difference of "vocabulary container", the div "term-container" isn't correctly displayed.

Example 1, correct (output of a page as http://www.mywebsite.com/taxonomy/vocabulary/1):

..<div class="vocabulary-container">
....<div class="vocabulary">
......<div class="description"></div>

......<div class="subterm-container">
........<div class="subterm">
..........<h2 class="title"></h2>
..........<div class="description"></div>
........</div>
......</div>
......<div class="subterm-container">
........<div class="subterm">
..........<h2 class="title"></h2>
..........<div class="description"></div>
........</div>
......</div>

....</div>
..</div>

Example 2, not correct (output of a term's page under any vocabulary):

..<div class="term-container">
....<div class="term">
......<div class="description"></div>
....</div>
..</div>

..<div class="subterm-container">
....<div class="subterm">
......<h2 class="title"></h2>
......<div class="teaser"></div>
....</div>
..</div>

..<div class="subterm-container">
....<div class="subterm">
......<h2 class="title"></h2>
......<div class="teaser"></div>
....</div>
..</div>

* Subterm's divs are'nt children of "term-container" (as happens with the div "vocabulary-container")

Thx in advance.

Comments

nedjo’s picture

nedjo
he/him/his
Primary language English
commented

Sure. A patch would be welcome.

duser’s picture

duser commented

It should be easy but unfortunately I don't know whether to do.

I think that this is the code that should be edited:

/**
 * Theme a term output.
 */
function theme_taxonomy_context_term($term) {
  $type = $term->subterm ? 'subterm' : 'term';
  $output = '';
  $output .= "<div class=\"$type-container\">\n";
  $output .= "  <div class=\"$type\">\n";
  if ($term->subterm) {
    $output .= '   <h2 class="title">' . l($term->name, taxonomy_term_path($term)) . "</h2>\n";
  }
  if ($term->teaser) {
    $output .= '   <div class="teaser">' . $term->teaser . "</div>\n";
  }
  else {
    $output .= '   <div class="description">' . $term->description . "</div>\n";
  }
  if ($term->links) {
    $output .= '   <div class="links">'. theme('links', $term->links) . "</div>\n";
  }
  $output .= "  </div>\n";
  $output .= "</div>\n";
  return $output;
}

And this is the code that generate the vocabulary container (correctly):

/**
 * Theme a vocabulary output.
 */
function theme_taxonomy_context_vocabulary($vocabulary) {
  $output .= "<div class=\"vocabulary-container\">\n";
  $output .= "  <div class=\"vocabulary\">\n";
  $output .= '   <div class="description">' . $vocabulary->description . "</div>\n";
  if ($vocabulary->links) {
    $output .= '   <div class="links">'. theme('links', $vocabulary->links) . "</div>\n";
  }
  if ($vocabulary->terms && is_array($vocabulary->terms)) {
    foreach($vocabulary->terms as $term) {
      $term->subterm = TRUE;
      $output .= theme('taxonomy_context_term', $term);
    }
  }
  $output .= "  </div>\n";
  $output .= "</div>\n";
 
  return $output;
}
nancydru’s picture

nancydru
she/her/hers
Primary language English
Location Boston
commented
Status: Active » Postponed (maintainer needs more info)

Is this still a problem with Version 2?