The Form Builder integration currently does some crazy work to pretend that pagebreak components aren't page breaks so that Webform will return the entire form with all pages in it. We don't need to have this special casing, since Webform already has a special $filter property that determines if field properties are sanitized or not. If $filter = FALSE, we can simply skip all conditional rules and page checking and return the entire form.

CommentFileSizeAuthor
webform_all_pages_unfiltered.patch1.79 KBquicksketch
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

quicksketch’s picture

Status: Needs review » Fixed

Committed to both 3.x branches.

emptyvoid’s picture

So, I have some crazy results.

On my local development environment the update doesn't appear to have any affect. However I had to manually delete the form_builder_cache entries for my webform for it to update correctly.

On my development server deleting the cache didn't help, rather when in "edit" mode the page break form elements disappeared and only the first page of fields is visible! I have confirmed that the components for all subsequent pages are GONE and are not within the database any longer. {crying in his soup}

Luckily I have a working copy locally so I'm going to manually add the component records back into the database.. lucky me :P

emptyvoid’s picture

Oh ok I updated webform to 7.x-dev March 29th 2012 release and the page break feature appears to "stick" and the fields don't get caplowied anymore.

quicksketch’s picture

Thanks for testing @emptyvoid! I didn't have any problems like you described, but it does require that you update both Webform and Form Builder at the same time.

Status: Fixed » Closed (fixed)

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