I do get the notice

Notice: Undefined index: rebuild in drupal_process_form() (line 927 of /includes/form.inc).

after installing the patch from http://drupal.org/node/1326716#comment-5657894.

It seems like the form submit mechanism in modal forms requires $form_state['rebuild'] to be present. Can we change this in webform_client_form_pages as per attached patch?

CommentFileSizeAuthor
webform-modal_forms-support.patch441 byteslarskleiner

Comments

larskleiner’s picture

larskleiner commented
Status: Active » Needs review
quicksketch’s picture

quicksketch
he/him
commented

Looks like this should be doable to me.

quicksketch’s picture

quicksketch
he/him
commented
Status: Needs review » Fixed

Changes didn't cause any problems for me (as I wouldn't think they would) so committed to both branches even though D6 doesn't really pay attention to $form_state['rebuild']; I included it for consistency.

quicksketch’s picture

quicksketch
he/him
commented
Title: Notice: Undefined index: rebuild in drupal_process_form() (line 927 of /includes/form.inc). » Change unset($form_state['rebuild']) to $form_state['rebuild'] = FALSE to prevent notices
Category: feature » task

Updating title to reflect commit message.

larskleiner’s picture

larskleiner commented

Great, thanks Nathan!

Status: Fixed » Closed (fixed)

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