Null values for merge vars break mailchimp validation [#2924936]

If no value is supplied for a webform field that is configured to be passed to mailchimp, then the following error is returned from the mailchimp API:

Data did not match any of the schemas described in anyOf

Comment	File	Size	Author
#3	2924936-null-values-for-merge-vars-3.patch	918 bytes	autopoietic
#2	2924936-null-values-for-merge-vars-2.patch	777 bytes	autopoietic

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Comment #1

20 November 2017 at 13:16

autopoietic created an issue. See original summary.

Comment #2

autopoietic CreditAttribution: autopoietic at Torchbox commented 20 November 2017 at 13:26

File	Size
2924936-null-values-for-merge-vars-2.patch	777 bytes

This patch converts the mergevars to strings before subscribing, which avoids problems with null values.

I am using 8.x-5.1, but I have rolled the patch against 8.x-5.x (currently 5.3).

Comment #3

autopoietic CreditAttribution: autopoietic at Torchbox commented 20 November 2017 at 13:41

File	Size
2924936-null-values-for-merge-vars-3.patch	918 bytes

Missed a line in patch

Comment #4

miro_dietiker

Switzerland

CreditAttribution: miro_dietiker at MD Systems GmbH commented 22 December 2017 at 13:22

Priority:

Normal

» Major

Promoting, can lead to data loss.

Comment #5

maweba CreditAttribution: maweba commented 28 March 2019 at 13:14

*push*

currently using hook alter

/**
 * Implements hook_mailchimp_lists_mergevars_alter().
 */
function MODULE_mailchimp_lists_mergevars_alter(&$mergevars, $webform_submission, $entity_type) {
  if(isset($mergevars) && count($mergevars)) {
    $mergevars = array_map('strval', $mergevars);
  }
}