Looking at the code, I think only one mailchimp list is saved when submitting the webform, whichever one comes last. I would like the ability to have multiple lists updated with the same form.

CommentFileSizeAuthor
#6 save_multiple_lists_on_submit-1185234.patch1.68 KBmattcasey
#5 save_multiple_lists_on_submit-1185234.patch1.68 KBmattcasey
#3 save_multiple_lists_on_submit-1185234.patch1.73 KBmattcasey
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

mattcasey’s picture

mattcasey CreditAttribution: mattcasey commented

Turned out to be pretty easy to fix, using an array instead of a string for $mailchimp_list. I'll try to submit a patch

jrockowitz’s picture

jrockowitz CreditAttribution: jrockowitz commented

mattwad, I just ran into this issue could you please post your fix (ie patch or code). Thanks.

mattcasey’s picture

mattcasey CreditAttribution: mattcasey commented
FileSize
save_multiple_lists_on_submit-1185234.patch1.73 KB

Here ya go.

jrockowitz’s picture

jrockowitz CreditAttribution: jrockowitz commented

The patch works great, thanks.

There is a very minor indentation issue but the maintainer should be able to commit it.

mattcasey’s picture

mattcasey CreditAttribution: mattcasey commented
Status: Active » Needs review
FileSize
save_multiple_lists_on_submit-1185234.patch1.68 KB

Rerolled with correct indentation and also removed a redundant comment.

mattcasey’s picture

mattcasey CreditAttribution: mattcasey commented
FileSize
save_multiple_lists_on_submit-1185234.patch1.68 KB

Gah! I missed one tab space.

blackdog’s picture

blackdog CreditAttribution: blackdog commented
Status: Needs review » Fixed

Good catch, committed to 6.x & 7.x. Someone please test the 7.x code.

Status: Fixed » Closed (fixed)

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