As MailMIME module depends on PEAR, basically it's incompatible with CiviCRM-enabled sites, also it seems to have issues with PHP 5.4 and later.
So I added a very simple plain text alternative that does not depend on mailmime module.

CommentFileSizeAuthor
#7 2484879-htmlmail-plain_text-6.patch2.13 KBaron novak
#5 2484879-htmlmail-plain_text-5.patch2.13 KBmr.york
#2 2484879-htmlmail-plain_text-2.patch2.11 KBaron novak
#1 2484879-htmlmail-plain_text-1.patch2.12 KBaron novak

Comments

aron novak’s picture

aron novak
Primary language Hungarian
Location Hungary, Budapest
commented
StatusFileSize
new 2484879-htmlmail-plain_text-1.patch2.12 KB
aron novak’s picture

aron novak
Primary language Hungarian
Location Hungary, Budapest
commented
StatusFileSize
new 2484879-htmlmail-plain_text-2.patch2.11 KB

Eh, content-type definition is fixed here.

salvis’s picture

salvis
he/his
commented

Thank you for your patch, Aron Novak — this looks reasonable and very useful!

Any reviewers/testers out there, please?

aron novak’s picture

aron novak
Primary language Hungarian
Location Hungary, Budapest
commented

https://www.drupal.org/u/mr.york pointed out that #2 can fail because of https://bugs.php.net/bug.php?id=68776 with PHP 5.x or newer
https://github.com/yohgaki/php-src/commit/89def6911128d8806fd10957a01833...

So under headers array, we should not add trailing newlines.

mr.york’s picture

mr.york commented
StatusFileSize
new 2484879-htmlmail-plain_text-5.patch2.13 KB

Attached EOL fix patch.

salvis’s picture

salvis
he/his
commented

Thank you for the improved patch, mr.york!

  1. +++ b/htmlmail.admin.inc
@@ -184,6 +184,16 @@ function htmlmail_admin_settings() {
+        '!mailmime' => l(t('Mailmime'), 'https://www.drupal.org/project/mailmime', array('attributes' => array('target' => '_blank'))),

    The module name is spelt "Mail MIME" and it must NOT be translated. 

  2. +++ b/htmlmail.mail.inc
@@ -120,6 +120,21 @@ class HTMLMailSystem implements MailSystemInterface {
+

    Please remove the empty line.

I need a third party to review and test this patch — anyone, please!

aron novak’s picture

aron novak
Primary language Hungarian
Location Hungary, Budapest
commented
StatusFileSize
new 2484879-htmlmail-plain_text-6.patch2.13 KB

Rerolled and applied the requested change. I am far from 3rd party, but FYI we have this in production for a few weeks without any complaints.

salvis’s picture

salvis
he/his
commented

Thanks!

What module(s) are you using for generating the emails? Are you by any chance using Subscriptions?

If you are, could this possibly be used for sending [subs:items:index:#0:formatted] as per #2534276: How to send HTML notifications?

  • Nafes committed 01c598b on 7.x-2.x authored by Aron Novak 
    Issue #2484879 by Aron Novak, mr.york: Provide simple plain/text...
Nafes’s picture

Nafes commented
Status: Needs review » Reviewed & tested by the community

Works perfectly for me. Just removed a redundant blank line near the end of changes. Thank you, Aron.

@salvis, can we mark the issue as fixed or you see some room for improvements?

salvis’s picture

salvis
he/his
commented
Assigned: aron novak » Unassigned
Status: Reviewed & tested by the community » Fixed

@Nafes: If it works, it's great!

I'm happy to see you come on board, but how did this happen? I don't see any issue about you getting maintainer access. Would you please post an issue with a few lines to introduce yourself, and also add your name on the front page? (and in Mail MIME, too)

Nafes’s picture

Nafes commented

As discussed, added my name on the front pages. Very happy to meet you, salvis!

Status: Fixed » Closed (fixed)

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