I want to display a node's taxonomy terms as a list (nd currently only has formatters to display them next to each other).

Would this be something you would consider including in nd? If so, I'll start a patch.

CommentFileSizeAuthor
#15 Schermafbeelding 2013-09-19 om 15.17.35.png67.27 KBbotris
#9 ndtermsaslist-1141524-3.patch1.75 KBbrunodbo
#3 ndtermsaslist-1141524-3.patch1.75 KBbrunodbo
#2 ndtermsaslist-1141524-2.patch2.45 KBbrunodbo

Comments

swentel’s picture

swentel commented

Very interested, even for D7 :)

brunodbo’s picture

brunodbo
he/him
commented
Status: Active » Needs review
StatusFileSize
new ndtermsaslist-1141524-2.patch2.45 KB

Following your approach for the other taxonomy term formatters (jeuj for copy/paste :)), this patch ...

1) Adds two new formatters to nd_ds_fields():

'nd_terms_per_vocabulary_list' => t('Unordered list'),
'nd_terms_per_vocabulary_linked_list' => t('Unordered list, linked to term'),

2) Passes these formatters in nd_theme() to nd_terms_per_vocabulary().

3) In nd_terms_per_vocabulary(), checks for the new formatters and outputs $terms as a list if $list is set.

brunodbo’s picture

brunodbo
he/him
commented
Version: 6.x-2.x-dev » 6.x-2.4
StatusFileSize
new ndtermsaslist-1141524-3.patch1.75 KB

Some irrelevant stuff in previous patch. Here's a new one.

Status: Needs review » Needs work

The last submitted patch, ndtermsaslist-1141524-3.patch, failed testing.

brunodbo’s picture

brunodbo
he/him
commented
Status: Needs work » Needs review

Hmm, not sure why testing fails, patch applies cleanly locally.

dave reid’s picture

dave reid
he/him
Primary language English
Location Nebraska USA
commented
Version: 6.x-2.4 » 6.x-2.x-dev

Trying against a branch rather than tag...

dave reid’s picture

dave reid
he/him
Primary language English
Location Nebraska USA
commented

#3: ndtermsaslist-1141524-3.patch queued for re-testing.

brunodbo’s picture

brunodbo
he/him
commented
Version: 6.x-2.4 » 6.x-2.x-dev

Shouldn't the re-test have run by now?

brunodbo’s picture

brunodbo
he/him
commented
StatusFileSize
new ndtermsaslist-1141524-3.patch1.75 KB

Uploading it again, to re-send it to testbot.

swentel’s picture

swentel commented
Project: Node displays » Display Suite
Version: 6.x-2.x-dev » 7.x-1.x-dev
Status: Needs review » Patch (to be ported)

Committed and pushed, thanks! IIRC I guess the test bot has troubles with dependency projects no ?
Moving to DS to see if I can reuse some of this stuff for taxonomy formatters in D7.

swentel’s picture

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

Marking this as won't fix, the field template option now has this option.

vasike’s picture

vasike
Location Ramnicu Valcea
commented
Category: feature » support
Status: Closed (won't fix) » Active

how does it works, as i can't this find this formatter for term reference.
What about having a comma separated terms.

heyyo’s picture

heyyo commented

Formatter for term reference works great to make an ordered list or unordered list but how could I add rel="tag" on the anchor HTML tag ?

swentel’s picture

swentel commented
Status: Active » Closed (fixed)

Going to add comma separated stuff in #1276966: View modes for taxonomy terms are inconsistently available for ds_field_formatter_info().

Edit: this already works with the expert field template. Extra attributes will be added soon.

botris’s picture

botris
Primary language Dutch
Location Creuse, France
commented
StatusFileSize
new Schermafbeelding 2013-09-19 om 15.17.35.png67.27 KB

I've attached a screenshot of settings on how to achieve this.
List settings for expert field settings