[wp-trac] [WordPress Trac] #23291: wp_mail should handle phpmailer exceptions instead of ignoring them
WordPress Trac
noreply at wordpress.org
Sun Jan 27 06:52:33 UTC 2013
#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
More information about the wp-trac
mailing list