drupal_mail uses old api

ouch - this will prevent error emails from going out it seems.

i went and fixed this without any testing. hope i got it right. it can't be any more broken. note that 6.x is not affected.

seems to work for me now - this issue was affecting my site...

thanks,

Adrian

The Drupal 6 branch *is* affected by this. Although it needs a different patch for http://api.drupal.org/api/function/drupal_mail/6

In my testing on d6, the error message was sent out to scores of invalid email addresses, with each word in the message considered a "To" address (most of them on the server's local domain) and an empty subject and body.

my fixes were to d6. are you sure you are using a version after my fixes? someone reported that hey did work.

According to http://drupal.org/cvs?commit=125745 the fix was on d5. I attached a patch for d6.

By the way this patch is a bit larger than it needs to be; I removed some newlines (\n) from the translatable strings since wrapping and concating of body sections can now be done by core mail functions. If it's important to avoid changing translatable strings I could reroll this.

At first glance, this looks good. I will review and commit soon.

Committed without any changes or testing. Please report back success or failure. Code looks good.