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

WordPress Trac noreply at wordpress.org
Thu Jan 30 08:51:25 UTC 2014


#25560: Update to PHPMailer 5.2.7
---------------------------------+------------------
 Reporter:  MattyRob             |       Owner:
     Type:  enhancement          |      Status:  new
 Priority:  normal               |   Milestone:  3.9
Component:  External Libraries   |     Version:  3.7
 Severity:  normal               |  Resolution:
 Keywords:  3.9-early has-patch  |     Focuses:
---------------------------------+------------------

Comment (by bpetty):

 Replying to [comment:12 SergeyBiryukov]:
 > FWIW, autoloader appears to be introduced as a part of a bigger commit
 (I didn't blame it properly in IRC):
 >
 https://github.com/Synchro/PHPMailer/commit/d992ae6dc4a14ccce7019d8a71d52b7f70d73403

 Yeah, took me a minute to track down too since it was rolled up with the
 PSR-2 changes. Mixing code format changes with actual functional
 changes... *sigh*

 @nacin, how about pushing to a subfolder, and still just sticking with the
 following files: LICENSE (since it is LGPL, not GPLv2 - probably should be
 included), class.phpmailer.php, class.smtp.php, and PHPMailerAutoload.php
 ? This would avoid any file changes from upstream, and even though we also
 strip extras, languages, POP3, and all other package files, it still
 benefits from being moved. I know the autoloader implementation is not
 ideal, but the only reason it's actually harmful is because we were
 already renaming files to begin with. This is WP's fault, not PHPMailer.
 It might be time to just fess up and fix it.

--
Ticket URL: <https://core.trac.wordpress.org/ticket/25560#comment:13>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform


More information about the wp-trac mailing list