To reproduce:

- Create a new form.
- Add new fields (text, file, etc.)
- Add a new fieldset.
- Drag the new fields into the new fieldset.
- Save.

Result:
- The new fields are saved, but not inside the fieldset.
- The fieldset is saved, but it is empty.

The new fields should be saved inside the fieldset.

CommentFileSizeAuthor
#2 form_builder_webform_new_fieldset-1795168-d6.patch2.57 KBquicksketch
#1 form_builder_webform_new_fieldset-1795168.patch2.59 KBquicksketch

Comments

quicksketch’s picture

quicksketch
he/him
commented
Status: Active » Needs review
StatusFileSize
new form_builder_webform_new_fieldset-1795168.patch2.59 KB

This bug was introduced in #1563710: Use node_save() directly to avoid conflicts with Entity Cache. Because we no longer save components individually as they are identified (we use node_save() instead all at once at the end), the CIDs cannot be retrieved from the database. Instead we should update the form cache as we assign CIDs to the components, so that subsequent children will be able to locate the parent CID. This patch provides this functionality.

quicksketch’s picture

quicksketch
he/him
commented
Status: Needs review » Fixed
StatusFileSize
new form_builder_webform_new_fieldset-1795168-d6.patch2.57 KB

Companion patch for D6.

Committed to both branches.

Status: Fixed » Closed (fixed)

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