[wp-trac] [WordPress Trac] #19549: Please remove X-Mailer from class-phpmailer

WordPress Trac wp-trac at lists.automattic.com
Sat Jun 2 13:26:05 UTC 2012


#19549: Please remove X-Mailer from class-phpmailer
-------------------------------------------------+-------------------------
 Reporter:  jwz                                  |       Owner:  westi
     Type:  enhancement                          |      Status:  assigned
 Priority:  normal                               |   Milestone:  Future
Component:  External Libraries                   |  Release
 Severity:  minor                                |     Version:  3.3
 Keywords:  has-patch commit 3.5-early 2nd-      |  Resolution:
  opinion                                        |
-------------------------------------------------+-------------------------
Changes (by clearsite):

 * cc: clearsite (added)
 * keywords:  has-patch commit 3.5-early => has-patch commit 3.5-early 2nd-
     opinion


Comment:

 If I may insert myself into this discussion, there is a way to do this
 that suits EVERYONE here.

 1. Create an extender; class wp_phpmailer extends phpmailer
 2. Create a new send function in there
 function Send() { do_action_ref_array( 'phpmailer_send', array( &$this )
 ); parent::Send(); }
 ->now we can hook just before sending, where we can alter all headers as
 much as we want.
 3. patch WordPress to use new wp_phpmailer instead of new phpmailer.

 This is all theory ofcourse

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


More information about the wp-trac mailing list