taxonomy_term_path returns bad alias

I think you have some other module that is messing up the path.

Take a look in your database, table "vocabulary", and see what "module" is set to for each vocabulary. My guess is that it's something other than "taxonomy" for the ones that gives you problem.

http://api.drupal.org/api/function/taxonomy_term_path/6

Also having a similar issue with my taxonomy URL aliases. Have tried version 1.1 and the latest dev version 2, both producing the same (unfriendly) output (e.g. 'http://localhost/taxonomy/term/31/all' - instead of 'http://localhost/taxonomy/term/green/all').

I have checked the table in question in the drupal database, and the module for all 10 of my taxonomies is set to 'taxonomy'.

Can @lutegrass's code suggestion by implemented, or is there a better way to fix this issue?

i have the same issue, aliases for taxonomy and vocabulary are ignores.

The only modules used for vocabulary are taxonomy and forum. Version 1.2

How about something like this? (site_map.module line 496 onwards)

    // Work out if the term path is being overridden.
    $raw_path = 'taxonomy/term/'. $term->tid;
    $term_path = taxonomy_term_path($term);
    // Compile the feed URL.  If the term path is not being overridden the
    // depth can be added.
    if ($raw_path == $term_path) {
      $feed_path = $term_path .'/'. $cat_depth .'/feed';
    }
    else {
      $feed_path = $term_path .'/feed';
    }
    // Only create an actual link if there is content tagged with this item.
    if ($term->count) {
      $term_item .= l($term->name, $term_path, array('attributes' => array('title' => $term->description)));
    }
    else {
      $term_item .= check_plain($term->name);
    }

Then change the $rss_link generation to:

      // Use the term_path generated earlier.
      $rss_link = theme('site_map_feed_icon', $feed_path);

(patch against DRUPAL-6--2 branch attached)

The correct patch is attached.

Subscribing

@DamienMcKenna
Thank for your patch

I've just tested #5 patch and here is feedback.

Taxonomy URL are properly written now.
But URL under Feed icons are always wrong.

Changed to BugReport.
Site_map seems not compatible with PathAuto in this particular case.

Is there any news ?

Really annoying because it duplicates all taxonomy URL for same term.
Result is very bad for SEO.

Very strange, problem has disappeared...
But there was no patch commited in Sitemap module since I've posted comments in this issue.

Maybe it has been fixed by recent updates in another module like PathAuto.

@schaub123
@wiifm69
@lasac
@DamienMcKenna
Do you still reproduce this issue ?

Subscribing

that worked for me! thanks for the patch

This is still happening in the latest version, 6.x-2.1 as well as 6.x-2.x-dev

The #5 patch:
Fixes taxonomy URLs.
Feed icons are still in the taxonomy/term/80/all/feed format.

I was able to edit the file successfully and can confirm taxonomy terms are now updating correctly with PathAuto settings.