Support: How-To Add XHTML to PHPTemplate's Primary Links Output?

There's always more than one way to skin a cat, as they say, and the same is true with your needs.

I don't know off-hand how to change the content of the $link function, though I'm sure it can be done.

What I've done is somewhat different. I've created classes that apply to the body tag of every page and then make my page and section styles happen from there on down... for example body.about or body.admin.

You can grab the code for this in the CivicSpace theme. Let me know if you have any questions.

So the short answer is that it's not readily possible to do what you want. This is the conversation I had with Steven:

factoryjoe: do you guys know how to pass HTML to the l() function?
for example if i want to output title?
how do i get the span in there?

killes
http://drupaldocs.org/api/head/function/l
l($text, $path, $attributes = array(), $query = NULL, $fragment = NULL, $absolute = FALSE, $html = FALSE)

factoryjoe:
can it be done @ the theme level?

UnConeD
factoryjoe: you need to pass $html as TRUE

factoryjoe:
then where does the HTML get put?
this is what i'm trying to answer: http://drupal.org/node/26846

UnConeD
in $text
the $html parameter decides whether what you pass is text or html
the reason it is like this is because it catches a lot of possible XSS issues
if you, then you're responsible for calling check_plain() on user-submitted data
*if you do

factoryjoe:
i c
so what's the answer to that person's question?

UnConeD
everything above is
pass $html = false to l(), wrap user-defined data in check_plain() in the $text parameter as this is no longer done for you, and add all the HTML you want

factoryjoe:
but how do you override <li><span><?php print $link ?></span></li>?

UnConeD
by overriding the themable function which generates $link

factoryjoe:
hmm
now i have to find that
isn't that a phptemplate function?

UnConeD
you want to add spans to primary links?

factoryjoe: apparently
didn't you read that guy's post?

UnConeD
no i'm sort of busy
this is what I use

function phptemplate_menu_item_link($item, $link_item) {
/* added magic spans for funky tabs */
return l('<span class="a"><span class="b">'. check_plain($item['title']) .'</span></span>', $link_item['path'], array_key_exists('description', $item) ? array('title' => $items['description']) : array(), NULL, NULL, FALSE, TRUE);
}

factoryjoe: ah ok

UnConeD
or maybe those are the regular tabs

factoryjoe:
yeah
those are your local tasks i think
not primary_links
i mean, the whole loop would need to be recreated, wouldn't it?
to break out the values contained in $link?

UnConeD
btw it seems those primary/secondary links are oddly enough generated in theme.inc's theme_get_settings
which would mean it's not themable
but i remember adrian saying he was rewriting some part of the links handling
in any case, this should really be a themable function, and the links should be passed as an array to the theme engine, not as prebaked links

yes its possible, and as said above one need to reproduce existing functions

i used another technique for my links, but needed the "active" info and special html layout too
see www.langmi.de (the tabbed primary links)

my code for this in the phptemplate (i deleted almost all nonrelevant functions)

function _phptemplate_variables($hook, $vars) {
/***********************************
 * HOOK PAGE
 * - frontpage
 *   - save 'which page' to the $vars 
 *   - override template file depending on page
 *   - add special stylesheets
 * - get the right is_front info, drupal core won't look at path aliases
 * - save some user rights info to the $vars
 *   - make customized primary links
 */
  if ($hook == 'page') {

    /**
      * FRONTPAGE
      */
    if ($vars['is_front']) {

      /**
        * save page info
        */
      $vars['my_page_type'] = 'front';
      
      // set frontpage layout 
      $theme = 'candy';
      $style = 'front';
      $page_type = 'frontpage';
      $vars = my_set_theme($theme, $page_type, $vars);
      $vars = my_add_style($theme, $style, $vars);

      /**
        * customized primary links
        * - see http://www.stunicholls.myby.co.uk/menus/snazzymenu2.html for more info on this link design
        */
      foreach (array('primary') as $type) {

        //Get the data to populate the textfields, if the variable is not an array .. try to parse the old-style link format.      
        $value = variable_get('phptemplate_' . $type . '_links', array());

        //Get the amount of links to show, possibly expanding if there are more links defined than the count specifies

        $count = variable_get('phptemplate_' . $type . '_link_count', 5);
        $count = ($count > sizeof($value['link'])) ? $count : sizeof($value['link']);

        if (theme_get_setting('toggle_' . $type . '_links')) {
          for ($i =0; $i < $count; $i++) {
            unset($attributes);
            if (!empty($value['text'][$i])) {
              if (!empty($value['description'][$i])) {
                $attributes['title'] = $value['description'][$i];
              }
              $text = $value['text'][$i];
              $link = $value['link'][$i];            
              // the original from phptemplate.engine changed to catch the produced links
              if (substr($link, 0, 4) == 'http') {
                $_templink = '<a href="'. $link .'"'. drupal_attributes($attributes) .'>'. $text .'</a>';
              }
              else {
              $_templink = l($text, $link, $attributes);
              }
              // reproduced "l" function from includes/common.inc to get the active link, only for "primary links"
              // classes are set acccording to my style.css, $classtext is needed for additional css logik
              if ($type == 'primary') {
                if (drupal_get_normal_path($link) == $_GET['q']) {
                  isset($attributes['class']) ? $attributes['class'] .= ' active' : $attributes['class'] = 'active';
                  $classtext = 'darkblue';
                }
                else {  
                  isset($attributes['class']) ? $attributes['class'] .= ' notactive' : $attributes['class'] = 'notactive';
                  $classtext = 'lightblue';
                }
                $_templink = '<a href="'. $link .'" class="menu"><b class="snazzy"><span class="boxcontent '. $attributes['class'] .'">'. $text .'</span><b class="b4 '.$classtext.'"></b><b class="b3 '.$classtext.'"></b><b class="b2 '.$classtext.'"></b><b class="b1"></b></b></a>';
              }
              $links[$type][] = $_templink;              
            }
          }
        }
        $vars['customized_type1_links']= $links['primary'];
      }
    }
  }

  /**
    * set front + site-mission according to path alias
    */
  if (drupal_get_path_alias($_GET['q']) == variable_get('site_frontpage', drupal_get_path_alias('node'))) {
    $vars['mission'] = theme_get_setting('mission');    
    $vars['is_front'] = true;
  }

  
  // calling function expects an array
  if (empty($vars)) { 
    $vars = array();
  }
  return $vars;
}

its not inside the phptemplate, but inside the template.php :-)

The original submitter asked for support, got it, and settled on a solution. Closing.