[wp-trac] [WordPress Trac] #25560: Update to PHPMailer 5.2.7

WordPress Trac noreply at wordpress.org
Sat Oct 19 16:44:00 UTC 2013


#25560: Update to PHPMailer 5.2.7
--------------------------------------+--------------------
 Reporter:  MattyRob                  |       Owner:
     Type:  enhancement               |      Status:  new
 Priority:  normal                    |   Milestone:  3.8
Component:  External Libraries        |     Version:  trunk
 Severity:  normal                    |  Resolution:
 Keywords:  dev-feedback 2nd-opinion  |
--------------------------------------+--------------------
Changes (by bpetty):

 * cc: bpetty (added)
 * milestone:  Awaiting Review => 3.8


Comment:

 I'll be back to this for a deeper review later here, likely adding some
 back compat API to go with this, but just some quick notes for now:

 (1) Yes, I think it's time to move this to it's own folder since we're
 being pushed to add new file(s) for the first time in years.

 (2) The only reason we keep renaming these files was for back compat
 purposes, and if this is moved to a new folder, we'll just end up either
 deciding to break this, or adding a bare placeholder include file in it's
 place. Either way, we should definitely stick with PHPMailer official
 filenames.

 (3) I think @nacin has expressed that there's not any issues with
 including the entire library as intended, and really it's not that big.
 I'd like more input from others here as well. I personally think we should
 just include all of it as intended.

--
Ticket URL: <http://core.trac.wordpress.org/ticket/25560#comment:1>
WordPress Trac <http://core.trac.wordpress.org/>
WordPress blogging software


More information about the wp-trac mailing list