Hi! I get the following if the cid is empty.
Notice: Undefined index: in _webform_client_form_submit_flatten() (line 2608 of
\sites\all\modules\webform\webform.module).
Notice: Undefined index: in webform_submission_data() (line 26 of
sites\all\modules\webform\includes\webform.submissions.inc).
Notice: Array to string conversion in drupal_write_record() (line 7069 of C:\web\docroot\s3stepitfi\docroot\includes\common.inc).
Notice: Array to string conversion in drupal_write_record() (line 7069 of C:\web\docroot\s3stepitfi\docroot\includes\common.inc).
| Comment | File | Size | Author |
|---|---|---|---|
| #2 | webform_client_form_submit_flatten-1910656.patch | 1.15 KB | quicksketch |
| #1 | webform_empty_cid_check_fix-1910656-1.patch | 1.58 KB | yce |
Comments
Comment #1
yce commentedSo I made a little fix for it.
Comment #2
quicksketchIt looks like #2008000: Check for $cid in webform_submission_data() to skip pseudo-fields took care of half of this issue, but it didn't make any adjustments to _webform_client_form_submit_flatten(). Based on your patch in #1, I adjusted _webform_client_form_submit_flatten() to skip elements that don't have real $cid values.
However considering the potential risk to existing users (and because the other issue only affected the new version), I've only committed this to the 7.x-4.x branch. This isn't really a bug in either version, since it's accommodating for "pseudo" elements being injected into the Webform form.