[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