[wp-trac] [WordPress Trac] #18792: Wrong FROM email when using wp_mail and built in mail() function

WordPress Trac noreply at wordpress.org
Tue Sep 17 09:56:49 UTC 2013


#18792: Wrong FROM email when using wp_mail and built in mail() function
-------------------------------------------------+-------------------------
 Reporter:  pigster                              |       Owner:  westi
     Type:  defect (bug)                         |      Status:  accepted
 Priority:  normal                               |   Milestone:  Awaiting
Component:  External Libraries                   |  Review
 Severity:  major                                |     Version:  3.2.1
 Keywords:  has-patch needs-testing needs-unit-  |  Resolution:
  tests 2nd-opinion                              |
-------------------------------------------------+-------------------------

Comment (by SergeyBiryukov):

 Replying to [comment:6 gruvin]:
 > As of release version 3.6.1, the patch from KevinHamilton above has
 apparently still not been implemented, though the code on pluggable.php
 has had new stuff addedd ...

 It's not a recent addition, see [4946] for #3862.

 > Has this issue been addressed already in development versions 3.7 or
 v3.8?

 Not yet, otherwise this ticket would already be closed.

 > How far away are those from release anyway? Where would I go to find
 that out myself?

 See http://make.wordpress.org/core/version-3-7-project-schedule/.

 > Can I contribute code or a patch directly, have it verified and included
 in v3.6.2 somehow?

 Minor releases are for security and regression fixes only. With enough
 testing, the patch can still make it into 3.7 or 3.8.

 > --OR-- Is wp_mail() deprecated anyway -- or should it be, in favor of
 using PHPMailer class directly in all plugins from now on?

 No, `wp_mail()` is not going to be deprecated.

 Replying to [comment:9 gruvin]:
 > OK. Here's a patch that takes care of all of the above.

 1. `class-phpmailer.php` is an external library, so we should avoing
 changing it unless it's something worth reporting upstream. Since all
 those properties are public, we can clear them in `wp_mail()` as needed.
 2. Please attach the patch as a file instead of pasting it in the comment
 box.

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


More information about the wp-trac mailing list