While reading through code and patches i've found a problem with simplemail.

A simplemail related issue in mimemail tries to change the api to make things work seamlessly:
http://drupal.org/node/400998

The solution would be very simple.
If a mail is HTML, it should call mimemail with $plain=NULL to allow mimemail to happen user consideration.
Since we call it with FALSE, user consideration is overridden.

Everything else is fine.

CommentFileSizeAuthor
simplenews_mimemail.patch1.03 KBmiro_dietiker

Comments

miro_dietiker’s picture

miro_dietiker
Location Switzerland
commented

correctly relating to case...
#400998: mimemail_prepare won't accept $plaintext = FALSE argument

miro_dietiker’s picture

miro_dietiker
Location Switzerland
commented
Status: Active » Closed (duplicate)

Hmm..
After realizing that there's one line more needed from an other patch i realized this bugreport was duplicate.
Redirecting work to:
#401014: Simplenews does not take the recipient's text mail preference into account