Primary-links issue

Something seems to be keeping this from working. AFAICT the snippet below is correct .. it is in my template.php I have tinkered with it some and flushed all caches plenty. It seems like something is preventing it from working.

<?php
/**
* Implementation of theme_menu_item_link()
*/

  function phptemplate_menu_item_link($link) {
    if (($link['menu_name'] == 'primary-links' || $link['menu_name'] == 'navigation')) {
      if (empty($link['localized_options'])) {
        $link['localized_options'] = array();
      }

      $profile = signwriter_load_profile('MenuLinkNormal');
      $link['localized_options']['html'] = TRUE;  // Because we'll be sending HTML into the l() call.
      return l(signwriter_title_convert($link['title'], $profile, array(), TRUE), $link['href'], $link['localized_options']);
    }
    else {
      return theme_menu_item_link($link);  // Use the inherited behavior for everything else.
    }
  }
?>

I have been able to replace block headers page titles and have some input filters working too so I know the module works.. its just the primary links wont change! Anyone have an idea what could be blocking it?