Hello

I have this form where there is a page break included, if you press "Next Page >" the user will see this PHP warning message:

"Warning: Invalid argument supplied for foreach() in _table_element_validate() (line 224 of /var/www/volcanoes/sites/all/modules/webform_table_element/webform_table_element.module)."

This will happen if you had an empty table in your form.

CommentFileSizeAuthor
#1 webform_table_element-adding_isset_to_remove_warning_message-2302687.patch701 bytesissa.haddadin
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

issa.haddadin’s picture

issa.haddadin CreditAttribution: issa.haddadin commented
Status: Active » Needs review
FileSize
webform_table_element-adding_isset_to_remove_warning_message-2302687.patch701 bytes

Here is a patch, i just added a simple isset()

Thanks.

issa.haddadin’s picture

issa.haddadin CreditAttribution: issa.haddadin commented
Version: 7.x-1.7 » 7.x-4.x-dev
issa.haddadin’s picture

issa.haddadin CreditAttribution: issa.haddadin commented
Issue summary: View changes
Issue tags: -PHP warnings (duplicate tag) +PHP warning message - Invalid argument supplied for foreach() in _table_element_validate()
issa.haddadin’s picture

issa.haddadin CreditAttribution: issa.haddadin commented
Priority: Normal » Minor
Issue summary: View changes
issa.haddadin’s picture

issa.haddadin CreditAttribution: issa.haddadin commented
Title: PHP Warning message » PHP Warning message - Invalid argument supplied for foreach() in _table_element_validate()
Issue tags: -PHP warning message - Invalid argument supplied for foreach() in _table_element_validate() +PHP warnings (duplicate tag)
Jochen Wendebaum’s picture

Jochen Wendebaum CreditAttribution: Jochen Wendebaum commented

I can confirm that this is working and think that it should be pushed into dev.