[wp-trac] [WordPress Trac] #30266: wp_mail() can send emails with a blank 'From' address

WordPress Trac noreply at wordpress.org
Fri Mar 20 15:07:53 UTC 2015


#30266: wp_mail() can send emails with a blank 'From' address
--------------------------+---------------------------
 Reporter:  drtonyb       |       Owner:  valendesigns
     Type:  defect (bug)  |      Status:  assigned
 Priority:  normal        |   Milestone:  4.2
Component:  Mail          |     Version:  4.0
 Severity:  normal        |  Resolution:
 Keywords:  needs-patch   |     Focuses:
--------------------------+---------------------------

Comment (by valendesigns):

 Replying to [comment:17 westi]:
 > * I think the unit tests would be better without the preg_matches and
 testing the actually generated headers - using the regular expressions
 looks strange in a set of tests to me

 I'll rethink that approach and update.

 > * I think that the change to add ```} elseif ( '' !== trim( $content ) )
 {``` should be commented so it is obvious why this is being done.

 Will add inline comments to the next iteration.

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


More information about the wp-trac mailing list