this module should use pathauto to create nice urls, ed classified is not yet ready, I will not use it.

Comments

milesgillham’s picture

Category: task » bug

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.

Cheers,

Miles

milesgillham’s picture

Status: Active » Closed (won't fix)

Flagging as won't fix unless further details.

weka’s picture

Status: Closed (won't fix) » Active

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

Node 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.

fgm’s picture

Title: path » 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
fgm’s picture

Version: 6.x-3.x-dev » 7.x-3.x-dev
Status: Active » Patch (to be ported)

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.

Cray Flatline’s picture

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.

Cray Flatline’s picture

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 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.

fgm’s picture

@CryAngel: feel free to suggest a patch :-)

desrosj’s picture

Version: 7.x-3.x-dev » 7.x-3.0-rc1

Subscribe.

fgm’s picture

@desrosj: it is no longer necessary to comment in order to subscribe to an issue: just click "Follow" on top.

jemisond’s picture

Issue summary: View changes

Three 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.

fgm’s picture

With this module end of life approaching along with Drupal 7 EOL, this issue is now moot.

fgm’s picture

Status: Patch (to be ported) » Closed (won't fix)