On line 65 in webform_mailchimp.inc there is a typo. Currently that line reads:

drupal_set_message(t('You do not have any Mailchimp lists defined.', 'error'));

But this is correct:

drupal_set_message(t('You do not have any Mailchimp lists defined.'), 'error');

Sorry I don't know how to create a patch file or I would!

CommentFileSizeAuthor
#3 webform_mailchimp-correct_line_65-1462018-2.patch434 bytestr33m4n
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

lsolesen’s picture

lsolesen CreditAttribution: lsolesen commented

You can read more about creating a patch here: http://drupal.org/node/707484

I would do the following to create a patch:

git clone --branch 7.x-1.x http://git.drupal.org/project/webform_mailchimp.git
git checkout 7.x-1.x
cd webform_mailchimp

Now you are in the working directory where you you can edit your files in whatever editor you prefer. When you edited the file:

git diff > [project_name]-[short-description]-[issue-number]-[comment-number].patch

Now you have your patch. Simply upload it backup.

Let me know if you have more questions.

tr33m4n’s picture

tr33m4n CreditAttribution: tr33m4n commented
Version: 7.x-1.0-alpha1 » 7.x-1.x-dev

Confirmed in 7.x-1.x-dev also

tr33m4n’s picture

tr33m4n CreditAttribution: tr33m4n commented
FileSize
webform_mailchimp-correct_line_65-1462018-2.patch434 bytes

Attached patch...

blackdog’s picture

blackdog CreditAttribution: blackdog commented
Status: Active » Needs review

Cool, will take a look a.s.a.p.

blackdog’s picture

blackdog CreditAttribution: blackdog commented
Status: Needs review » Fixed

Thanks, committed!

Status: Fixed » Closed (fixed)

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