From: Evan Prodromou Date: Tue, 21 Jun 2011 18:07:10 +0000 (-0400) Subject: Add tags and headers to improve email summary deliverability X-Git-Url: https://git.mxchange.org/?a=commitdiff_plain;h=718203306c6187250ba53ec5c41d0db4ba08d6b4;p=quix0rs-gnu-social.git Add tags and headers to improve email summary deliverability SpamAssassin doesn't like HTML email without a) a Mime-Version header and b) an tag in the body. Although not everyone uses SA, there are probably other spam systems that implement similar rules. And it's always nice to play nice. --- diff --git a/plugins/EmailSummary/useremailsummaryhandler.php b/plugins/EmailSummary/useremailsummaryhandler.php index f03e8a1dcd..61542e4f3b 100644 --- a/plugins/EmailSummary/useremailsummaryhandler.php +++ b/plugins/EmailSummary/useremailsummaryhandler.php @@ -120,8 +120,16 @@ class UserEmailSummaryHandler extends QueueHandler $new_top = $notice->_items[0]->id; } + // TRANS: Subject for e-mail. + $subject = sprintf(_m('Your latest updates from %s'), common_config('site', 'name')); + $out = new XMLStringer(true); + $out->elementStart('html'); + $out->elementStart('head'); + $out->element('title', null, $subject); + $out->elementEnd('head'); + $out->elementStart('body'); $out->elementStart('div', array('width' => '100%', 'style' => 'background-color: #ffffff; border: 4px solid #4c609a; padding: 10px;')); @@ -201,19 +209,18 @@ class UserEmailSummaryHandler extends QueueHandler common_config('site', 'name'))."

"); $out->elementEnd('div'); + $out->elementEnd('body'); + $out->elementEnd('html'); $body = $out->getString(); // FIXME: do something for people who don't like HTML email - $subject = sprintf(_m('Your latest updates from %s'), common_config('site', 'name')); - - // TRANS: Subject for e-mail. - mail_to_user($user, $subject, $body, - array('Content-Type' => 'text/html; charset=UTF-8')); + array('Content-Type' => 'text/html; charset=utf-8', + 'Mime-Version' => '1.0')); if (empty($ess)) { $ess = new Email_summary_status();