Make WordPress Core

Opened 15 years ago

Closed 15 years ago

#19677 closed enhancement (fixed)

Update phpMailer to version 5.2

Reported by: westi Owned by: westi
Priority: normal Milestone: 3.4
Component: External Libraries Version: 3.3
Severity: normal Keywords: has-patch commit
Cc: Focuses:

Description

PHPMailer development is now in Apache Extras and the current version is 5.2

We should update this and then push any changes we want to recommend upstream

Improvements in 5.2:

  • Adds support for more MIME content types

Attachments (2)

phpmailer-5.2.diff (42.6 KB ) - added by westi 15 years ago.
PHP 5.2 for WordPress
19677.diff (566 bytes ) - added by MattyRob 15 years ago.

Download all attachments as: .zip

Change History (10)

@westi
15 years ago

PHP 5.2 for WordPress

#1 @westi
15 years ago

With the update to PHPMailer 5.2 we no longer need any hacks in the Unit Tests to make the code testable so I removed them for WP 3.4 and later in [UT492]

#2 @westi
15 years ago

In [19632]:

Update PHPMailer to version 5.2. See #19677

#3 @westi
15 years ago

Leaving the ticket open in case these changes bring any issues - not expected as they passed all our unit-tests :)

#4 follow-up: @barry
15 years ago

http://core.trac.wordpress.org/browser/trunk/wp-includes/class-phpmailer.php?rev=19632#L760

Looks broken. The - should probably not have been changed to a .

Not sure how the unit-tests passed :)

#5 in reply to: ↑ 4 @westi
15 years ago

Replying to barry:

http://core.trac.wordpress.org/browser/trunk/wp-includes/class-phpmailer.php?rev=19632#L760

Looks broken. The - should probably not have been changed to a .

Not sure how the unit-tests passed :)

Good Catch.

Core WordPress always uses mail() sending not smtp which is why the tests didn't catch this.

@MattyRob
15 years ago

#6 @MattyRob
15 years ago

  • Keywords has-patch added

Here's a quick patch for the class-smtp.php file name difference. This is how it was implemented in previous versions.

#7 @SergeyBiryukov
15 years ago

  • Keywords commit added

#8 @ryan
15 years ago

  • Resolutionfixed
  • Status newclosed

In [19642]:

Include class-smtp.php not class.smtp.php. fixes #19677

Note: See TracTickets for help on using tickets.