[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