It appears this module is not compatible with CCK's theming standard (field.tpl.php), instead relying on hardcoded
's on lines 230 and 238 of content_taxonomy.module

If I'm wrong on this or there is a way to get this working, any feedback or documentation on this would be much appreciated as I'm currently having to directly edit the formatting in content_taxonomy.module, something I've managed to avoid from doing with any other module so far.

CommentFileSizeAuthor
#3 contentax1.patch2.27 KBOzeuss

Comments

Raving’s picture

Raving commented

Sorry, that's "hardcoded <br />'s" :)

budda’s picture

budda
Primary language English
Location Cheshire
commented

Yes those pesky hardocded BR tags are quite annoying, can they be slipped in to a theme function so i can get rid of them neatly?

Ozeuss’s picture

Ozeuss commented
Status: Active » Needs review
StatusFileSize
new contentax1.patch2.27 KB

The problem lies in the fact that content_taxonomy uses the load $op in hook_field but not the view $op. So, data is loaded differently, but cck loads the field as default, which requires the "for each" loops in the field_formatter function.
This patch adds a view $op to the hook_field function, and iterates it correctly- sends a 'tid' variable to the formatter function, which then loads the term object. in other words- the for each loop in field_formatter is 'bumped' into content_taxonomy_field, and is called the number of objects loaded by the module.
Seems to work for me, please test.

Raving’s picture

Raving commented

Seems to work for me, on the latest version, too :)

ckng’s picture

ckng
Primary language English
commented

The patch makes some fields not displaying on certain view.
Simply removing the <br /> does not seems to cause other problem.

anthonym’s picture

anthonym commented

I second the suggestion to make this a themeable function. Just removing the line break might suit every purpose. Comma-separated terms would be nice for instance... Any ideas on how to accomplish this?

smitty’s picture

smitty commented
Status: Needs review » Fixed

Seems to be fixed in version of 2008-Jul-07.

Anonymous’s picture

Anonymous (not verified) commented
Status: Fixed » Closed (fixed)

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