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

Comments

mattcasey’s picture

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 commented

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

mattcasey’s picture

mattcasey commented
StatusFileSize
new save_multiple_lists_on_submit-1185234.patch1.73 KB

Here ya go.

jrockowitz’s picture

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 commented
Status: Active » Needs review
StatusFileSize
new save_multiple_lists_on_submit-1185234.patch1.68 KB

Rerolled with correct indentation and also removed a redundant comment.

mattcasey’s picture

mattcasey commented
StatusFileSize
new save_multiple_lists_on_submit-1185234.patch1.68 KB

Gah! I missed one tab space.

blackdog’s picture

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.