In the dart_taxonomy module I want to be able to inject all of a terms ancestors in the key|value pairs on an ad tag. I believe this should be done with an option on the tag to enable it.

So for example if I give a node a category of 'US Business' which is in the category branch, Business > World Business > US Busniess, all three terms will appear in the tag.

Attached is a patch for the latest dev version.

CommentFileSizeAuthor
#3 dart_taxonomy_ancestor_terms-1356050-3.patch1.72 KBdwkitchen
#2 dart_taxonomy_ancestor_terms-1356050-2.patch1.95 KBjamiecuthill
dart_taxonomy_ancestor_terms.patch1.94 KBjamiecuthill

Comments

bleen’s picture

bleen commented

I would be in favor of adding this ... but I'd love to get #1355836: Trying to get property of non-object in dart_taxonomy_dart_key_vals() in first since this will likely need to be rerolled at that point

jamiecuthill’s picture

jamiecuthill commented
StatusFileSize
new dart_taxonomy_ancestor_terms-1356050-2.patch1.95 KB

Here is a new patch following #1355836

dwkitchen’s picture

dwkitchen commented
StatusFileSize
new dart_taxonomy_ancestor_terms-1356050-3.patch1.72 KB

Here's a revised patch for the current dev version

bleen’s picture

bleen commented
Status: Active » Reviewed & tested by the community

I think this looks good ... Ill be committing this later tonight :)

bleen’s picture

bleen commented
Status: Reviewed & tested by the community » Fixed

committed here: http://drupalcode.org/project/dart.git/commit/1f37c9f

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.