After evaluating the Mailout module it seems that a Content-Type is passed from MimeMail which is not checked against yet in smtp.module.
Currently the smtp.module checks content-type against 'multipart/mixed' and 'multipart/alternative'.
Mailout sends the preformatted body labelled as 'multipart/related'

I suppose it should be checked against that Content-type as well, modifying line 238 to:

            else if ((strpos($lvalue, 'multipart/mixed') !== FALSE) || (strpos($lvalue, 'multipart/alternative') !== FALSE) || (strpos($lvalue, 'multipart/related') !== FALSE)) {

avoids the problem as sending the preformatted body as plain text. I don't know whether this is only the behavior of the Mailout module or Mimemail's, but it's definitely worth checking against. This fix is really small, but code needs review as I don't know all the content-types possible, maybe there are more which define preformatted body.

CommentFileSizeAuthor
smtp_mailout.patch828 bytesmelon

Comments

berenddeboer’s picture

berenddeboer commented

Second this patch. Had to do exactly the same thing. Please make a new release with this patch.

oadaeh’s picture

oadaeh commented

Hmm. I thought I had replied to this issue already, but I guess not.

I won't be fixing this issue in that way. I have been using the 6.x branch to test removing the PHPMailer code from the module, and it looks like it might be okay (except for maybe a bug or two). Until I backport the changes, this is a fine work around, but I won't be including it in the code. I'll be doing the same thing as the 6.x branch.

melon’s picture

melon commented
Status: Needs review » Postponed

Thanks for the reply. Then I suppose it should be postponed until the backports.

wundo’s picture

wundo commented
Issue summary: View changes
Status: Postponed » Fixed

Mimemail works now ;)

Status: Fixed » Closed (fixed)

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