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)
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

ezra-g’s picture

ezra-g CreditAttribution: 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
CreditAttribution: sun commented
Status: Needs review » Reviewed & tested by the community
FileSize
cck-DRUPAL-5.formatter.patch737 bytes

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

sun’s picture

sun
Primary language German
Location Karlsruhe
CreditAttribution: sun 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 CreditAttribution: 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
he/they/他
Primary language English
Location Taipei, Taiwan
CreditAttribution: JohnAlbin 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
he/they/他
Primary language English
Location Taipei, Taiwan
CreditAttribution: JohnAlbin 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 CreditAttribution: yched commented
Status: Reviewed & tested by the community » Fixed

Committed. Thx all.

Anonymous’s picture

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

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