Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
Thank you Kristof! for create new branch to support cck multigroup 6.3-dev. Here is my rough patch to make it works with Display Suites 6.x-2.x-dev branch and Node Display Contribute 6.x-3.x-dev branch.
Comment | File | Size | Author |
---|---|---|---|
#6 | ds_multigroup_support-1206622-6.patch | 14.55 KB | jstoller |
#5 | ds_multigroup_support-1206622-5.patch | 14.63 KB | jstoller |
#1 | ds_patch_support_multigroups-1206622-1.patch | 12.56 KB | vtsan |
ds_patch_support_multigroups.patch | 12.56 KB | vtsan |
Comments
Comment #1
vtsan CreditAttribution: vtsan commentedHere is the right patch file name
Comment #2
jstollersubscribe
Comment #3
AnybodyThank you very very much for the patch, I was just wondering that nothing worked anymore since the last update. Works great for my quite huge case.
I also anounced your patch here: http://julian.pustkuchen.com/drupal-cck3-multigroup-display-suite-ds-tab...
Comment #4
AnybodySorry but I still have a little problem after patching: The Field labels don't show up. Fieldgroups work, but CCK labels went away. Just the ":" shows up in front of the field value in full node view.
Any ideas on that or can anyone confirm?
Comment #5
jstollerI went through Vincent's patch and compared it to the current 6.x-1.x-dev version. It looks like there were a lot of unnecessary discrepancies, likely do to older workarounds and such. This patch should bring this version closer to that version, with the addition of multigroup specific changes. At the very least, I think it is a cleaner starting point. You might consider committing it for that reason alone. It mostly works! ;-)
This patch has fixed a problem I was having with fieldsets not rendering properly. However, I too am still having an issue with missing field labels. For some reason the $field['title'] variable is empty by the time it gets to the theme functions. I haven't been able to trace back where the problem lies though. Any suggestions would be appreciated.
Comment #6
jstollerHere's a minor update to the patch, removing some code junk I just noticed.
The label issue seems to affect the 6.x-1.x-dev version as well. It looks like the latest devs are only setting that title variable for ds fields and ignoring cck fields. I'll keep looking into it.
@Anybody: can you test this patch and let me know if you see any other problems with it? If not, then we should try to get it committed soon, as a step in the right direction. It will make it easier for the maintainers to incorporate fixes which go into 6.x-1.x-dev.
Comment #7
Anybody@jstoller: Thank you very much. I will try to test it today, currently I am very busy. I'll post my feedback here asap!
Comment #8
Anybody@jstoller: Works great! Just the missing cck titles are really bad. I'm sorry that I haven't got enough time currently to check it out myself.
I hope that will be resolved soon.
Comment #9
jstoller@Anybody:
Thanks for testing the patch. I'm marking this RTBC. I've filled a separate issue for the CCK label problem: #1214052: Label and complete cck theme flow missing on CCK fields.
@swentel:
If you're following this, can you please commit this patch. It may not be perfect, but it gets things (mostly) working again and gives us a cleaner baseline to develop from.
Comment #10
swentel CreditAttribution: swentel commentedCommitted this, thanks!