Make WordPress Core

Opened 17 years ago

Closed 17 years ago

Last modified 17 years ago

#3862 closed defect (bug) (fixed)


Reported by: ryan's profile ryan Owned by: ryan's profile ryan
Milestone: 2.2 Priority: normal
Severity: normal Version: 2.1.1
Component: Administration Keywords: mail smtp encoding wp_mail
Focuses: Cc:


Integrate phpmailer to gain smarter header encoding and the option of SMTP. We could setup phpmailer to use php's mai() by default and add a hook that would allow plugins to configure phpmailer for SMTP.

phpmailer will hopefully fix encoding issues like #209.

Attachments (1)

phpmailer.diff (84.8 KB) - added by ryan 17 years ago.
class-phpmailer.php, class-smtp.php, new wp_mail()

Download all attachments as: .zip

Change History (8)

17 years ago

class-phpmailer.php, class-smtp.php, new wp_mail()

#1 @yskin
17 years ago

Looks nice. Good news to me. Many people, include me, can't use PHP's mail() function on their virtual hosts.

I remember that PHPMailer doesn't support SSL connection so it can't connect Google SMTP server.

#2 @rob1n
17 years ago

Patch looks good, +1 from me for including phpmailer. That way we don't have to deal with all that mail stuff in the core.

#3 @rob1n
17 years ago

  • Cc rob1n added

#4 @ryan
17 years ago

I found this thread on adding SSL support.

#5 @ryan
17 years ago

  • Resolution set to fixed
  • Status changed from new to closed

(In [4946]) Trying out phpmailer for wp_mail(). fixes #3862

#6 @robertaccettura
17 years ago

Should note that phpMailer doesn't seem to handle 8bit emails with utf-8 encoded subjects very well. They seem to get truncated.

#7 @ryan
17 years ago

I've been looking over swiftmailer, but it is has dozens of files versus phpmailer's one file. If anyone wants to experiment with plugging swiftmailer into wp_mail(), let us know how it goes.

Use their php4 package rather than the php5 one.

Note: See TracTickets for help on using tickets.