as the subj says: need path_auto/url_alias support for this module... I like it!

CommentFileSizeAuthor
#14 nocatdepth.20070111.patch1.7 KBjoestewart

Comments

frjo’s picture

Category: feature » support

That how it works today, at lest on the sites where I use the site map module. See the demo site for an example.

Comperare the links you see in the site map and the ones you have built aliases for. My guess there are some differences between them like "taxonomy/term/15/0" vs "taxonomy/term/15/all" or similar.

Christoph C. Cemper’s picture

Hu?

>My guess there are some differences between them like "taxonomy/term/15/0" vs "taxonomy/term/15/>all" or similar.

I want links that are like

articles/topics/jewelry

and no internal term ids etc...

frjo’s picture

Example: Existing system path "taxonomy/term/15/0" is aliased to "articles/topics/jewelry". Site map has in turn been told to put out "taxonomy/term/15/all" and there is no alias for that.

The fix is to change the site map settings or change the alias.

taxonomy/term/15/0 mean show all nodes belonging to term 15 (this is the default so taxonomy/term/15 works also)
taxonomy/term/15/all mean show all nodes belonging to term 15 and all nodes belonging to any child terms of term 15

Christoph C. Cemper’s picture

Frjo /// thanks for explanations...

I guess I'm just dumb... I generated all aliases from scratch and it worked... forgot that some of them werre created by INSERT INTO :-)

thanks,christoph

frjo’s picture

Status: Active » Fixed
Anonymous’s picture

Status: Fixed » Closed (fixed)
TheWhippinpost’s picture

Title: Site_map cool, but taxonomy url_aliases shall be used instead of the orignal ones » No url_aliases on Taxonomy Resulting in Dupe Content in Google...
Version: » 4.7.x-1.x-dev
Category: support » bug
Status: Closed (fixed) » Active

I know you've suggested an answer to this but I'm not sure I'm understanding you, or the solution you proposed wouldn't be practical if importing lots of new node and category content.

If I understand you correctly, you are suggesting that the administrator manually alias (for example):

taxonomy/term/30/all >> articles/topics/jewelry

Is that right?

If I'm correct, unless there's a way to automate pathauto to create such relationships, then I'm not sure this module is suitable for a site-owner transfering an existing site with lots of content, or one that grows quite rapidly, as it would incur a huge amount of work to manually create aliases.

... unless I'm missing something!

As it stands, site-map is printing links to category as described above (taxonomy/term/30/all) which means that search engine spiders essentially record 2 different paths to the same content, resulting in a duplicate content issue/penalty.

(For the record, my pathauto alias is setup to output category paths as: articles/topics/jewelry/index.htm.)

Leaving that aside, for search-engine-friendliness (and arguably user-friendliness too), it would be a better solution to have the site-map output ALL links to all nodes/content types, rather than a narrow sub-selection of books and one menu (for reasons that would take-up another thread to explain) (Google in particular, only spiders in-depth if the site has enough pagerank "juice", so it's better practice to have as many deep-links near to the surface as possible).

I think that whilst having links-to-category is a useful option (subject to the paths resolving "correctly"), it would be better served as a supporting option, secondary to having all pages linked, by default (and if necessary, continuing on page 2, page 3 etc... for large sites).

Anyway, I may well be experiencing a "blind-spot" which is preventing me seeing the solution rigt now - though I have looked - in which case, I apologise for re-opening this issue. Otherwise, I would describe it as a bug on account it really should output all links consistent with the aliased path structure (without having to further manually intervene) and that it harms a site in terms of SEO

frjo’s picture

Component: Code » Documentation
Category: bug » support

If you have alias set up, be it manually, via pathauto or whatever, site map module will use them, you don't need to do anything.

As I said in comment #3 users need to understand that e.g. "taxonomy/term/15/0" are different from "taxonomy/term/15/all". By default site map will generate the later but you can change that in the settings.

TheWhippinpost’s picture

Thanks for your prompt reply frjo.

I think you'd be as well to explain where the settings are that control this behaviour seeing as a few people have come up against this. I'm looking but can't for the life of me see a setting on Admin >> Settings >> Sitemap.

I thought (clutching at straws) you might be referring to changing the "Categories depth:" from "All", to a specific depth but that results in the same behaviour.

Pretend I'm a 6 year old.

TheWhippinpost’s picture

Well (for the possible benefit of others facing the same issue), after setting 'RSS feed depth' to 0 (zero), and after several forced refreshes of the browser, the RSS links seem to be now using pathauto's alias, as desired.

(Cache is off BTW)

... but still no change with the category links.

frjo’s picture

Take a look at "Categories depth" settings, it work the same as "RSS feed depth".

Make also sure you have the latest 4.7 version of the module, v 1.18.2.1 2006/08/24.

frjo’s picture

Status: Active » Fixed
Anonymous’s picture

Status: Fixed » Closed (fixed)
joestewart’s picture

StatusFileSize
new1.7 KB

I too had this same issue. pathauto does not generate url_alias with any depth after the term id. Using the attached patch, choosing a negative number for cat_depth now does not append any cat depth and site_menu generates an url like taxonomy/term/term_id which is recognized as an alias correctly for me.

I did not patch the same for RSS feeds.

This may not be applicable for everyone but it fixed things for me.

Joe

frjo’s picture

Title: No url_aliases on Taxonomy Resulting in Dupe Content in Google... » Add option to remove depth from taxonomy links
Version: 4.7.x-1.x-dev » master
Component: Documentation » Code
Assigned: Unassigned » frjo
Category: support » feature
Status: Closed (fixed) » Fixed

Committed patch in #14 from joestewart to HEAD.

Anonymous’s picture

Status: Fixed » Closed (fixed)
Anonymous’s picture

Version: master » 5.x-1.2
Status: Closed (fixed) » Active

I would like to reopen this issue, following on from Joe's comment in posting #15:

I did not patch the same for RSS feeds.

I cannot seem to find a way to make pathauto generated aliases for RSS feeds work with Site map. Setting the "RSS feed depth" to "all" or "0" does lead to broken links (actually, they are not broken, they just do not lead to the RSS feed) - and removing them by inputting "-1" as with the categories does not work. It only seems to accept all or 0.

Would it be possible to change the behaviour of RSS feed depth in the same way as for the categories?

frjo’s picture

Status: Active » Fixed

TfR75, I think you are using Views example taxonomy view instead of Drupal built in function. Disable the View or make it work with depth and I believe the problem will go away.

Anonymous’s picture

Category: support » feature

frjo: thank you very much for you reply. You are right, there is a view in play here (a default Drupal view called "The taxonomy view with a depth of 0.").

However, I am not sure if this is the issue: it does not matter what depth I enter for the view or if I leave that field blank within the view, Site map always adds either "all" or "0" to the RSS path, depending on its own configuration settings. Furthermore, the view itself displays RSS feeds that work exactly as they are supposed to. I have also deleted the view, but that changed nothing. The only setting that seemed to have any influence is the depth setting for RSS within Site map - but that only allows "0" and "all".

I have of course cleared views cache after making these changes, but nothing helped.

frjo’s picture

Category: feature » support
Status: Fixed » Active

In my opinion that example taxonomy view is broken and should not be used, as anything besides an example.

Simply disable it and Drupal will use the normal built in display instead. Then your problem will go away.

Anonymous’s picture

Category: feature » support

Sadly, it does not. As outlined above, I deleted the view, emptied the views cache and the problem persisted. Any other ideas?

Anonymous’s picture

Okay, so I have done this again, i.e. deleted the view, emptied the views cache, emptied the Drupal and browser cache etc. and now the RSS feeds are working again. Sadly though, they do still not use the alias but show /term/TERMID/all/feed. Any ideas?

frjo’s picture

What path does pathauto generate the RSS alias for? Have you rebuilt them after you turned off the view?

Drupal core builds up term RSS links like this:

http://example.com/taxonomy/term/123/[depth]/feed

Where depth can be "all", or a number from 0 and up.

Site map has a setting to choose what depth to use.

Anonymous’s picture

Thanks for continuing to look into this, frjo!

No, I had not rebuilt the aliases, but I have done so now - no change.

We use aliases for all content on the site, using [vocab-raw]/[catpath-raw] for all category path settings with feed added for the feeds. The non-aliased paths are of the type /taxonomy/term/3277/all/feed.

frjo’s picture

The non-aliased paths are of the type /taxonomy/term/3277/all/feed.

Then you need to set RSS feed depth to "all" on the Site map settings page (admin/settings/sitemap).

Site map will then output path like /taxonomy/term/3277/all/feed and since Site map run it through the Drupal function url() it will be converted to the alias if it exist.

Anonymous’s picture

frjo, I had tried that earlier on and it is still not working. With the view disabled the RSS link from sitemap does work again, but it is still not aliased.

frjo’s picture

Is the original term path that pathauto has generated aliases the same as Site map outputs?

If it's and you still have problems I'm at loss how to help you.

Anonymous’s picture

Yes, Site Map shows the original path as created by Drupal, it just refuses to display the alias. Well, thanks for your help anyway - it is really great that you took the time to look into this!

frjo’s picture

Status: Active » Fixed

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.

gemstone’s picture

Title: Add option to remove depth from taxonomy links » Peridot Gemstone

If you want to keep your Peridot GemStone new. Don’t place peridot jewelry in an ultrasonic cleaner. The changes of temperature can cause peridot to crack. Always keeps your stone in a tarnish resistant bag. It will keep your Peridot gem shinier and grace. http://www.peridotlovers.com