Make catalog and breadcrumb links work with pathauto for SEO

Posts: 41
Joined: 09/07/2007

I've noticed that the [tid] is used in the paths for all of the catalog links. The function uc_catalog_term_path() puts the term ID in the path. If uc_catalog_term_path() just used the taxonomy term path, pathauto can be used for SEO and user friendly links. I have edited uc_catalog_term_path() to this:

function uc_catalog_term_path($term){
  if (function_exists('iconv')){
    $path = "taxonomy/term/" . $term->tid;
    return $path;
  }
}

This works fine for all of the links in the navigation block and breadcrumbs, the only problem that I have now is getting the navigation block to know what which categories to expand and the breadcrumbs leave off the catalog link on some of the catalog pages. The navigation block could be fixed in this function _uc_catalog_navigation(), but that's beyond my abilities.

I search through the site and found this post that says it used to work this way, but has been changed.
http://www.ubercart.org/forum/development/767/catalog_paths_and_7c

Is there any plans for this to work better with pathauto in the future?