Reviewed & tested by the community
Project:
Content Construction Kit (CCK)
Version:
6.x-2.x-dev
Component:
Views Integration
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
25 Mar 2010 at 00:19 UTC
Updated:
23 Dec 2014 at 18:26 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
codigovision commentedmoving to views, if this is not posted in the correct place, please direct me in the right direction to get this issue resolved, thank you.
Comment #2
merlinofchaos commented1) Support requests should not be marked critical
2) How is Views even involved here?
Comment #3
codigovision commentedSorry, I had this as a CCK Issue, but no responses, and it just occurred to me that the problem is only when the field is displayed in a view. I just need to add a wrapper for the item when there is only one.
Comment #4
codigovision commentedfound the issue in cck/includes/views/handlers/content_handler_field_multiple.inc there is an if statement when rendering the field, and if there is only one value it does not get themed. I changed it to this for now, just to get it to work, but I realise it is a poor hack:
Personally, as someone who creates and themes a lot of Drupal sites, I think it should always render themed.
Any thoughts on this? Is this something that can be updated to head? If not how can i write this into my theme?
Thank you,
www.codigovision.com
Comment #5
shaneonabike commentedI agree... and actually it makes it super hard to customise your own code to deal with this properly
Comment #6
shaneonabike commentedI applied the suggested workaround and it seems to have resolved the problem for me as well. I'll try and roll a patch in the next day or so here.
Comment #7
shaneonabike commentedEeesh my bad this is over a year due :(
Please test and let me know :)
Comment #8
codigovision commentedYeah looks like a cleaner solution. As soon as I get a chance I'll apply it and post back. Thanks!
Comment #9
shaneonabike commentedCan we apply this just so folks who are still using CCK-6 can have the latest changes... I can't upgrade until this is pushed.
Comment #10
shaneonabike commented@codigo: ping?
Comment #11
codigovision commentedHi Shane, Sorry I never posted back. I have since upgraded the project in question to Drupal 7 and haven't had an issue with this since. Not sure if this ever made it into version 6.
Comment #12
shaneonabike commentedStill an issue but the patch is still legit!