WebformMailChimpHandler does not check whether fields that are submitted with the webform have a corresponding merge tag. Where they do not this results in a 'badly formed resource' response from Mailchimp, and the respondent is (silently) not subscribed.

CommentFileSizeAuthor
#3 webform_mailchimp-check_merge_tags-2872572-3.patch1.66 KBautopoietic

Comments

autopoietic created an issue. See original summary.

autopoietic’s picture

autopoietic commented
Title: Does not subscribe webform field element does not have matching merge tag in mailchimp » Does not subscribe if no matching merge tag in mailchimp for field element
autopoietic’s picture

autopoietic commented
StatusFileSize
new webform_mailchimp-check_merge_tags-2872572-3.patch1.66 KB

This patch checks the merge tags that are available before calling mailchimp_subscribe, and only sends those that have a corresponding merge field.

This makes it possible to have some fields that are sent to mailchimp, and others that are not (and so are just collected by webform as normal).

autopoietic’s picture

autopoietic commented
Status: Active » Needs review
hanoii’s picture

hanoii
Primary language Spanish
Location 🇦🇷UTC-3
commented
Parent issue: » #2883538: Several improvements / Request for co-maintainership

A different approach for this is included in #2883538: Several improvements / Request for co-maintainership among other features. If possible a try of that patch could help get most features in.

hanoii’s picture

hanoii
Primary language Spanish
Location 🇦🇷UTC-3
commented
Status: Needs review » Fixed

I am closing this one as the parent issue is now in.

Status: Fixed » Closed (fixed)

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