The attached patch adds support for Views and token to Primary Term, as well as fixing a node_load()/node_save()-related bug.
| Comment | File | Size | Author |
|---|---|---|---|
| #8 | primary_term-160382-8.patch | 4.23 KB | jonskulski |
| #5 | 160382.patch | 3.29 KB | mvc |
| primary-term-views-token.patch | 6.25 KB | bjaspan |
Comments
Comment #1
bjaspan commentedComment #2
(not verified) commentedComment #3
spiffyd commentedDoes this patch work for 6.x? Can someone roll this into CVS..
Comment #4
summit commentedSubscribing, greetings, Martijn
Comment #5
mvcThe port to Drupal 6 and Views 2 used the default views taxonomy handlers, meaning that the views field shows all available taxonomy terms for the given node. I've written a custom field handler which returns only the primary term, just like the module did in 5.x. (It can't be filtered by vocabulary, but as there's only ever one primary term I feel this is sufficient.) Please review!
Comment #6
spearhead93 commented160382.patch did the work for me. mvc you saved me a couple of hours of dev of this one.
I'll add more detailed review as I'll implement it in a quite specific environment (primary term + multilingual website). Code is correct.
Comment #7
jonskulski commentedJust going to add my +1 here and say the patch worked great.
Comment #8
jonskulski commentedExtended the patch to be able to link correctly to the term page.
Ideally we'd be working more off the views_taxonomy handlers, but for now I needed to patch it so here it is.
Comment #9
sethcohn commented+1 to this
Comment #10
drifter commented#8 works as advertised.
Comment #11
sethcohn commentedthanks Drifter! I'll be rolling up a new version shortly. Finally getting done with a big golive push, and getting breathing space again.
Comment #12
brianV commentedDrifter's patch in #8 committed:
http://drupal.org/cvs?commit=435648