Regardless of what formatter you set for the field, CCK's panels integration will always pass the full formatter string from panels, which is both field name and formatter in a string. e.g. field_name:full. Because this isn't a valid formatter for any field, the formatter ends up being "default."

This patch passes the value for $formatter that is derived earlier in the function call.

CommentFileSizeAuthor
#2 cck-DRUPAL-5.formatter.patch737 bytessun
cck-panels-formatter.patch738 bytesAnonymous (not verified)

Comments

ezra-g’s picture

ezra-g commented

I diagnosed this problem unaware that this issue existed in the queue. I can confirm that the change made by this patch does in fact solve this issue. I'll let the maintainers mark RTBC.

sun’s picture

sun
Primary language German
Location Karlsruhe
commented
Status: Needs review » Reviewed & tested by the community
StatusFileSize
new cck-DRUPAL-5.formatter.patch737 bytes

Same issue here, and 1:1 the same patch.

sun’s picture

sun
Primary language German
Location Karlsruhe
commented
Title: CCK's content_panels_render_field always passes an invalid formatter » content_panels_render_field always passes default formatter
gracearoha’s picture

gracearoha commented

Hi,

I was having the same problem with CCK 1.7. I tried the patch and all works fine with default, bold and italic settings. But when i try to use "trimmed", i get no text at all. I have tried trimming at 200 and 400 characters with no luck. Any ideas where i might look would be much appreciated.

johnalbin’s picture

johnalbin
they/他
Primary language English
Location Taipei, Taiwan
commented

k. Looks like I'm the 4th person to independently build the exact same patch. :-p

This patch is so RTBC, it's crazy.

johnalbin’s picture

johnalbin
they/他
Primary language English
Location Taipei, Taiwan
commented
Priority: Normal » Critical

BTW, if you are trying to use panels context and pull an image field into a pane, this bug causes it to not be rendered at all. Because the default formatter doesn't work on image/file fields. So its a critical patch for those types of fields.

yched’s picture

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

Committed. Thx all.

Anonymous’s picture

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

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