I've been using ed_classified with pathauto just fine. It shows up in the pathauto configuration tab as below:
Default path pattern for Classified Ad (applies to all Classified Ad node types with blank patterns below):
ads/[mm]/[title-raw]
And the ads appear to abide by that spec. Can you provide a little more detail as to the problem you're experiencing? Latest versions of Drupal 6 core, Pathauto, Classified Ads? I'm changing the request to a bug report as the functionality is definitely there, I just want to understand what is not working for you.
Pathauto is working fine for aliasing the node and taxonomy paths.
Is there a way to change the URLs in the links generated by the /ed-classified page(node)? The links to categories on that page are still created as example.org/ed-classified/tid/72
» Provide customizable paths to Classified category pages
Version:
6.x-2.0-alpha5
» 6.x-3.x-dev
Category:
bug
» feature
Priority:
Normal
» Minor
Rewording the issue, Classified Ads defines category pages at URLs like ed_classified/tid/(tid) in 6.x-2.x and classified/(tid) in 6.x-3.x/7.x-3.x and build such links on the ed_classified (*-2.x) or classified (*-3.x) page.
It could be useful for SEO purposes if the module:
could allow direct redefinition of these URLs in the module
or provided token definitions and a section for pathauto
Note that
this is a feature request, not a bug
these links can be customized on a per-site basis by implementing hook_menu_alter() to modify the module URLs, so downgrading to minor
This already works in Classified Ads 6.x-3.x with pathauto 6.x-2.x: just define a pattern for the Classified Ads vocabulary, and it will generate the needed aliases every where, thanks to the hook_term_path() implementation.
So this is fixed for Drupal 6.
Drupal 7 works differently and no longer has hook_term_path(), so needs some more work, as per http://drupal.org/update/modules/6/7#hook_url_outbound_alter. On the other hand, it has the Classified Link field formatter, so this is less necessary.
I think it's a really important issue for the 7.x branch. Vocabulary path should not be like classified/380, it should have path alias generated by pathauto. It's desperately needed for SEO.
Please change priority to this task. :) Thanks in advance.
I've looked to the module, and (as I see), that rewrited paths only used to output subcategories and slightly change query to output node teasers on the term page. And because of that we have ugly URLS without any chance to change it to our needs.
I've set a new alias with Pathauto and deleted/bulk updated multiple times, yet the taxonomy terms still go to classified/tid. This is not at all friendly, especially with a hierarchical taxonomy.
Comments
Comment #1
milesgillham commentedI've been using ed_classified with pathauto just fine. It shows up in the pathauto configuration tab as below:
ads/[mm]/[title-raw]
And the ads appear to abide by that spec. Can you provide a little more detail as to the problem you're experiencing? Latest versions of Drupal 6 core, Pathauto, Classified Ads? I'm changing the request to a bug report as the functionality is definitely there, I just want to understand what is not working for you.
Cheers,
Miles
Comment #2
milesgillham commentedFlagging as won't fix unless further details.
Comment #3
weka commentedPathauto is working fine for aliasing the node and taxonomy paths.
Is there a way to change the URLs in the links generated by the /ed-classified page(node)? The links to categories on that page are still created as
example.org/ed-classified/tid/72Node path settings:
[vocab-raw]/[term-raw]/[title-raw]Taxonomy term path settings:
[vocab-raw]/[cat-raw]I am running Classified Ads 6.x-2.0-alpha6 on Drupal 6.15 with Pathauto 6.x-1.2.
Comment #4
fgmRewording the issue, Classified Ads defines category pages at URLs like ed_classified/tid/(tid) in 6.x-2.x and classified/(tid) in 6.x-3.x/7.x-3.x and build such links on the ed_classified (*-2.x) or classified (*-3.x) page.
It could be useful for SEO purposes if the module:
Note that
Comment #5
fgmThis already works in Classified Ads 6.x-3.x with pathauto 6.x-2.x: just define a pattern for the Classified Ads vocabulary, and it will generate the needed aliases every where, thanks to the
hook_term_path()implementation.So this is fixed for Drupal 6.
Drupal 7 works differently and no longer has hook_term_path(), so needs some more work, as per http://drupal.org/update/modules/6/7#hook_url_outbound_alter. On the other hand, it has the Classified Link field formatter, so this is less necessary.
Comment #6
Cray Flatline commentedI think it's a really important issue for the 7.x branch. Vocabulary path should not be like classified/380, it should have path alias generated by pathauto. It's desperately needed for SEO.
Please change priority to this task. :) Thanks in advance.
Comment #7
Cray Flatline commentedI've looked to the module, and (as I see), that rewrited paths only used to output subcategories and slightly change query to output node teasers on the term page. And because of that we have ugly URLS without any chance to change it to our needs.
I suggest to use hook_query_alter() as described here http://drupal.org/node/1136520 in the last comment.
This way we can change query (if we really need it). Subcategories can be printed from taxonomy term template file.
Comment #8
fgm@CryAngel: feel free to suggest a patch :-)
Comment #9
desrosj commentedSubscribe.
Comment #10
fgm@desrosj: it is no longer necessary to comment in order to subscribe to an issue: just click "Follow" on top.
Comment #11
jemisond commentedThree years and no patch yet for 7.x branch??
I've set a new alias with Pathauto and deleted/bulk updated multiple times, yet the taxonomy terms still go to classified/tid. This is not at all friendly, especially with a hierarchical taxonomy.
Comment #12
fgmWith this module end of life approaching along with Drupal 7 EOL, this issue is now moot.
Comment #13
fgm