<span> In Primary Links

Caroline, your explanation was very clear and easy to follow, but after I tried to apply this method to do the exact same thing on my site (i.e. simply add span tags to each of the primary links), my menu simply disappeared.

I've tried a dozen different approaches, and one of two things happens: the menu either disappears entirely from the source code, or the menu formats the same as it always did.

I tried starting very simple, just adding the function override to my template.php file and adding an HTML comment so that I could see that the PHPTemplate engine was reading the template.php override, but no dice. To all appearances, the engine is ignoring my template.php file -- at least, that's my best theory.

I'm incredibly frustrated. I've quadruple-checked spellings, filenames, where I'm uploading to, making sure I have the opening and closing PHP tags: ...

Any ideas? Anyone?

Beth

This solution http://drupal.org/node/130177#comment-237501 worked for me in Drupal 5.

-------------------------------------------------
AdAstra.ca - Optimze the Web. Optimze the World.

In Drupal 6, if I use:

... phptemplate_links(.....

It works fine to theming all links as Carolina said, but if I rename the function like this:

... phptemplate_primary_links(.....

everything disappear!
(yes, I use "...theme('primary_links',..." as explained in page.tpl )

I think the reason is that in Drupal 6 I have to register the new function in template.php, but I can't understand how this can be done! I need an example please! I read i have to use "hook_theme()" to do so, but... HOW!

Please give me an example.

Thanks

Massimo

Make a template.php file in your theme folder.

This code will add a ~ between list items:

<?php
$my_delimeter = "~";

function phptemplate_primary_links($links, $attributes = array('class' => 'links')) {
  $output = '';

  if (count($links) > 0) {
    $output = '<ul'. drupal_attributes($attributes) .'>';

    $num_links = count($links);
    $i = 1;

    foreach ($links as $key => $link) {
      $class = '';

      // Automatically add a class to each link and also to each LI
      if (isset($link['attributes']) && isset($link['attributes']['class'])) {
        $link['attributes']['class'] .= ' ' . $key;
        $class = $key;
      }
      else {
        $link['attributes']['class'] = $key;
        $class = $key;
      }

      // Add first and last classes to the list of links to help out themers.
      $extra_class = '';
      if ($i == 1) {
        $extra_class .= 'first ';
      }
      if ($i == $num_links) {
        $extra_class .= 'last ';
      }
      $output .= '<li class="'. $extra_class . $class .'">';

      // Is the title HTML?
      $html = isset($link['html']) && $link['html'];

      // Initialize fragment and query variables.
      $link['query'] = isset($link['query']) ? $link['query'] : NULL;
      $link['fragment'] = isset($link['fragment']) ? $link['fragment'] : NULL;

      if (isset($link['href'])) {
        $output .= l($link['title'], $link['href'], $link['attributes'], $link['query'], $link['fragment'], FALSE, $html);
      }
      else if ($link['title']) {
        //Some links are actually not links, but we wrap these in <span> for adding title and class attributes
        if (!$html) {
          $link['title'] = check_plain($link['title']);
        }
        $output .= '<span'. drupal_attributes($link['attributes']) .'>'. $link['title'] .'</span>';
      }

      $output .= "</li>\n";
      
      if ($i < $num_links) {
          $output .= $my_delimeter;
      }
      

      $i++;
    }

    $output .= '</ul>';
  }

  return $output;
}
?>

Apparently, I almost had the solution by myself, but the check_plain() stuff was not something I would have figured out on my own. That comment shows a very simple solution and provides a great explanation of how it works.