Skip to content

GitLab

C
Core
Closed
Opened by beto_aveiga@beto_aveiga
Report abuse New issue

Avoid overwrite only first field with last value for fields with multiple values

Overview

Webforms with multiple emails values are not being saved correctly. The last submitted value is written to the first email field. Other emails are not being updated.

Before

  1. Data before submission image

  2. Example configuration of the webform image

  3. Data when visiting the webform image

  4. Filling the webform with a second email image

  5. Visiting the webform again we can see that first email was overwritten with last email image

  6. From the Admin UI it confirms what the webform was showing: first email overwritten with last email. image

After

  1. Repeat steps 1 to 4 from "Before" for initial data, filling the webform and saving.

  2. Data was stored correctly. image

Technical Details

  • Attaching exported webform with node export: webform-multiple-emails.txt

  • Site created for testing with civibuild using this command civibuild create civi12 --type drupal-demo --civi-ver 4.7.27 --url http://civi12 --web-root /home/beto/buildkit/build/civi12

  • CiviCRM currently uses civicrm_webform 7.20, however this module was replaced with the last version found here https://github.com/colemanw/webform_civicrm

Comments

Base on the modifed code (shown below) this is probably not only applicable to emails but for fields which are not addresses. image

Edited by beto_aveiga
To upload designs, you'll need to enable LFS. More information