[wp-trac] [WordPress Trac] #23291: wp_mail should handle phpmailer exceptions instead of ignoring them

[WordPress Trac]

#23291: wp_mail should handle phpmailer exceptions instead of ignoring them
-------------------------+------------------------------
 Reporter:  mark-k       |       Owner:
     Type:  enhancement  |      Status:  new
 Priority:  normal       |   Milestone:  Awaiting Review
Component:  Mail         |     Version:  3.5
 Severity:  minor        |  Resolution:
 Keywords:  has-patch    |
-------------------------+------------------------------

Comment (by iandunn):

 @mark-k, I don't see the point in only calling trigger_error() if WP_DEBUG
 is enabled. PHP's error_reporting already allows people to filter what
 kinds of messages they want. I think the standards practice is to push all
 errors/warnings/notices, and let the user pick which ones they want to
 display/log in a given context.

 @rmccue, Adding a @wp_error param sounds like a good idea to me, but we'd
 still need to trigger_error(), unless you're suggesting we change all the
 core calls to wp_mail() to use the new param. Otherwise exceptions
 triggered by core calls to wp_mail() would still be ignored.

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