Check markup only before wrapping

I am experiencing the same issue and it's a real downer....any suggestions on how could i fix it?

edit: i forgot to point out that i am experiencing it on the latest release ie beta2

subscribing

I think we should move the check_markup() call to hook_mail() instead of hook_mail_alter() which is called for all messages. Please test the attached patch and report back.

#506112: wrong line wrapping on drupal_wrap_mail() in mail.inc is marked as a duplicate of this.

However, we need to remove the check_markup() call from drupal_mail_wrapper().

In the following patch I moved the check_markup() call from drupal_mail_wrapper() into hook_form_alter(). We need this to create HTML versions of plain text system messages, but we need to check if the message needs to be converted to HTML format or not.

Lets move this to D7.

patch in #7 works great for me. Mime mail wasn't converting line breaks to br tags at all before implementing this patch.

Code looks good. Have not verified that it fixes the bug.

I would like to confirm the patch in #7 works for me too. I've patched against Mime Mail 6.x-1.0. The HTML portion of the email now has complete lines that aren't broken in the browser except for intended breaks inserted with ckeditor.

I implemented the exact same solution before I found this post! It would be nice if the patch could eventually be included with the 6.x branch. I think this would be classified as a bug, not a feature. I have several annoyed end-users complaining about weird wrapping HTML email. I'll be manually applying this patch to fix the issue for now.

#7 does not apply to 6.x-2.x or 6.x-2.0.

When will this make it into 6.x?

How about 7.x? I'm trying to get something like #7 for 7.x but I can't figure out where the second part of the diff for mimemail.module would go.

In order to handle at least the mails of user.module in Drupal 7, I came up with this hook_mail_alter() implementation:

function mymodule_mail_alter(&$message) {
  $mail_system = drupal_mail_system($message['module'], $message['key']);
  $is_mimemail_active = ($mail_system instanceof MimeMailSystem);
  if (
    $is_mimemail_active
    &&
    ('user' == $message['module'])
    &&
    in_array($message['key'], array(
      'register_admin_created', 'register_no_approval_required',
      'register_pending_approval', 'password_reset', 'status_activated',
      'status_blocked', 'cancel_confirm', 'status_canceled',
    ))
  ) {
    $message['body'][0] = nl2br($message['body'][0]);
  }
}

Where "mymodule" would be the name of a custom module of your own. Yep, it's just a work-around, not a real solution.

Thanks @raphaelhuefner for this work-around.