Hi,
Frist, thank you for this great tool. This was something crucially missing from drupal. This module worked except for it also changed my titles for all the nodes (articles - from a new CCK) as well. So not only did I have the new term title "new expanded term title" and new term meta tag, all the nodes listed titles were changed to this new name.

I am a newbe and a non coder (rely on the kindnes of others here) but was able to make a minor change to the template.php file (listed for this module) so this would not happen. Please see below and let me know if this look ok to you: All I Change was the reference of title to tax_title

// for TAXONOMY TITLE
if (arg(0) == 'taxonomy' && arg(1) == 'term' && is_numeric(arg(2)) && arg(2) > 0){
// taxonomy term page - check for custom page title
$tax_title = _taxonomy_title_get_title(arg(2));
if ($tax_title != ''){
// sets the meta title
drupal_set_title($tax_title);
// assures the heading is also set
$vars['tax_title'] = $tax_title;
}
}

Comments

scrambled’s picture

Category: bug » support
jenlampton’s picture

Hi scrambled,

I suspect that what's happening is that you forgot the switch statement supplied in the example code.

Simply changing the variable from $vars['title'] to $vars['tax_title'] will just stop the new custom titles from being printed in your template where you print the $title.

Please cut and paste the whole _phptemplate_variables function into your template.php and try again. Let me know if you still have issues.

Jen

jenlampton’s picture

Scrambled,
I know why you're confused, the template.php that got rolled into this module was not the example I intended to show you. I have updated it to the correct version with a sample of code that looks like this :

/**
 * Override or insert PHPTemplate variables into the templates.
 */
function _phptemplate_variables($hook, $vars) {
  switch ($hook) {
    case 'page':

    // for TAXONOMY TITLE
    if (arg(0) == 'taxonomy' && arg(1) == 'term' && is_numeric(arg(2)) && arg(2) > 0){
      // taxonomy term page - check for custom page title
      $title = _taxonomy_title_get_title(arg(2));
      if ($title != ''){
        // sets the meta title
        drupal_set_title($title);
        // assures the heading is also set
        $vars['title'] = $title;
      }
    }
    // for TAXONOMY TITLE

    break;
  }
  return $vars;
}

Sorry for the confusion!

jenlampton’s picture

Status: Active » Fixed

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.