Hi,

I have downloaded and installed your module. It works perfectly with urls such as "/node/xx". I tried to work with taxonomy term urls and it seems that their urls are not being replaced by their aliases. (/taxonomy/term/8 -> /my-alias).

Are you planning on implementing such feature?

Thanks, Martin.

CommentFileSizeAuthor
#4 1173368-convert-taxonomy-term-urls-1.3.patch10.63 KBjesse.d
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

LoMo’s picture

LoMo CreditAttribution: LoMo commented

It shouldn't be too hard to implement aliasing of term IDs. It just wasn't a part of my original use case, so I didn't think of it. Currently the module only looks up aliases for nodes, but I hope it's not too hard to do the same for terms. Views might be a bit trickier, but I hope to manage them, too. I just need to find a quiet weekend to get my head thoroughly comfortable with Git... at least enough not to worry about messing everything up, and then commit some of the improvements I've already made and add this feature, too -- I agree, it should handle your use case, too.

Thanks for pointing out this "to-do". :-)

LoMo’s picture

LoMo CreditAttribution: LoMo commented

See #1758394: hide bad filter hides good taxonomy term path, marked as duplicate of this issue — of course term paths should not be unlinked by the "hide bad" filter, either.

jesse.d’s picture

jesse.d CreditAttribution: jesse.d commented

I'm about to start working on supporting taxonomy term urls for a client project. I'll provide a patch when I have something.

jesse.d’s picture

jesse.d CreditAttribution: jesse.d commented
Version: 7.x-1.1 » 7.x-1.3
FileSize
1173368-convert-taxonomy-term-urls-1.3.patch10.63 KB

Patch adds taxonomy term path support to title filter and fixes removal of taxonomy term paths in the hide bad filter.

jesse.d’s picture

jesse.d CreditAttribution: jesse.d commented
Status: Active » Needs review
capfive’s picture

capfive CreditAttribution: capfive commented
Issue summary: View changes

ran into a need for this this morning, wondering why this never moved forward? I'll apply patch and post my results :)

capfive’s picture

capfive CreditAttribution: capfive commented
Status: Needs review » Reviewed & tested by the community

Applied patch and it is working well! :) haven't seen any issues yet, thank you!

capfive’s picture

capfive CreditAttribution: capfive commented

anyone going to commit this? I have to apply it to every site I use taxonomy on lol

MPeli’s picture

MPeli CreditAttribution: MPeli commented

Could anyone commit this please? Thank you!

philsward’s picture

philsward CreditAttribution: philsward commented

I think this need re-rolled against dev