Closed (won't fix)
Project:
Classified Ads
Version:
7.x-3.0-rc1
Component:
Code
Priority:
Minor
Category:
Feature request
Assigned:
Unassigned
Reporter:
Created:
26 Jul 2009 at 18:31 UTC
Updated:
9 Apr 2024 at 13:31 UTC
Jump to comment: Most recent
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