If using the panels node add/edit form template with the generic "node being edited" base form, field groups are not rendered or rendered in an unexpected fashion.

horizontal tabs => did not display
vertical tabs => did display, but the form was duplicated above
fieldsets => did not display

I'm not sure if this would be a field_group, ctools, or panels issue.

CommentFileSizeAuthor
#4 run-forms-pre_render-functions-before-rendering-1226034-4.patch1.16 KBarnested
#4 run-field_groups-field_group_build_pre_render-1226034-4.patch1.1 KBarnested

Comments

nils.destoop’s picture

nils.destoop commented
Project: Field Group » Chaos Tool Suite (ctools)
Version: 7.x-1.0 » 7.x-1.0-rc1

Moving this to ctools module. All groups are correctly prepared, but get lost when entering "ctools_form_content_type_render"

uprojects’s picture

uprojects commented

same problem !Have find a solution please ?

bochen87’s picture

bochen87 commented

big problem! also field groups do not appear as selectable field in both the form and the node

arnested’s picture

arnested commented
Status: Active » Needs review
StatusFileSize
new run-field_groups-field_group_build_pre_render-1226034-4.patch1.1 KB
new run-forms-pre_render-functions-before-rendering-1226034-4.patch1.16 KB

We also ran into the problem of not having fieldgroups rendered.

We "solved" the problem by ensuring that field_group_build_pre_render() is run in ctools_form_content_type_render() (patch: run-field_groups-field_group_build_pre_render-1226034-4.patch).

We also implemented it in a more generalized way by ensuring that a forms #pre_render functions are always run in ctools_form_content_type_render() (patch: run-forms-pre_render-functions-before-rendering-1226034-4.patch).

Whether either of these is the right approach I'm not sure as I find it hard to know the consequences of running pre_render functions at this stage.

Arne

hydra’s picture

hydra commented
Status: Needs review » Reviewed & tested by the community

Works fine. (first patch)

jherencia’s picture

jherencia commented

I have the same problem and the first patch fixes this.

raulmuroc’s picture

raulmuroc commented

First patch fixes it. ;) Let's commit?

merlinofchaos’s picture

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

I went ahead and committed the more generalized version based on several RTBCs.

Status: Fixed » Closed (fixed)

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