Thanks for the module, exactly what I am looking for, but I have an issue. Not matter if the block is set to ID or Name, the link is always the name, which leads to a page not found message.

CommentFileSizeAuthor
#3 taxonomy_accordion.module.txt5.2 KBlordofthelake

Comments

-enzo-’s picture

-enzo- commented

Hello john

Can you show me a example of the link you are getting?

if you can gave access to your drupal instance can be great.

Thanks.

enzo

john.campbell-higgens’s picture

john.campbell-higgens commented

You can take a look at my test site - www.equinelist.co.uk

If you click on the location block on the right, you can click on a location, but you always get the link of the name http://www.equinelist.co.uk/term/Warrington instead of the ID.

This gives me a page not found error.

lordofthelake’s picture

lordofthelake commented
Status: Active » Needs review
StatusFileSize
new taxonomy_accordion.module.txt5.2 KB

I need this module for my site, so fixed this and a couple of other bugs myself.
Please review the corrected code. If it's fine, I'd be glad if it could be merged upstream.

Thank you and regards.
- M.

-enzo-’s picture

-enzo- commented
Status: Needs review » Fixed

Hello John

Thanks for your report I just fix the problem.

Lordofthelake I didn't see your file before my path, but looks like is fixed now with my new version, next time please send a patch to help to apply changes quick.

Download the veriwon 6.x.1.1 http://ftp.drupal.org/files/projects/taxonomy_accordion-6.x-1.1.tar.gz

Regards,

enjoy

lordofthelake’s picture

lordofthelake commented

I haven't tested your code yet, but I think that line 131, in taxonomy_accordion.module, should be changed from

      'arguments' => array('vid' => NULL,'term' => NULL,'children'=>NULL,'base_path' => NULL)

to

      'arguments' => array('vid' => NULL,'term' => NULL,'children'=>NULL,'base_path' => NULL, 'term_link' => NULL)

or $term_id will not be passed to the callback (resulting in links in the form path/$id even when path/$name should be used instead.
Let me know if this makes sense for you.

Regards,
-- M.

-enzo-’s picture

-enzo- commented
Status: Fixed » Closed (fixed)

Hello lordofthelake

OK that is only for theme declaration is a mistake , but not a php error, but was fixed and included in version 1.2

Regards,

enzo