For some reason when creating campaigns using mailchimp_campaign_send_campaign() the call to getCampaign() reported the status as MAILCHIMP_STATUS_SENDING rather than MAILCHIMP_STATUS_SENT, and this caused the function to report the send had failed failed when in fact it hadn't.

I changed

if ($campaign->status == MAILCHIMP_STATUS_SENT) {
to
if ($campaign->status == MAILCHIMP_STATUS_SENT || $campaign->status == MAILCHIMP_STATUS_SENDING) {

to work around this.

Does it make sense, and is MAILCHIMP_STATUS_SENDING an OK status at that point?

CommentFileSizeAuthor
#2 2871536-2.patch766 bytessamuel.mortenson
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

hughworm created an issue. See original summary.

samuel.mortenson’s picture

Status: Active » Needs review
FileSize
766 bytes

That does make sense, could you test that this patch works for you?