[wp-trac] [WordPress Trac] #28319: Add hook to WP_Error for logging and report purposes
WordPress Trac
noreply at wordpress.org
Mon Aug 4 20:44:56 UTC 2014
#28319: Add hook to WP_Error for logging and report purposes
-----------------------------------------+-----------------------------
Reporter: ebinnion | Owner: johnbillion
Type: enhancement | Status: reopened
Priority: normal | Milestone: Future Release
Component: General | Version: trunk
Severity: normal | Resolution:
Keywords: 2nd-opinion has-patch close | Focuses:
-----------------------------------------+-----------------------------
Changes (by rmccue):
* keywords: 2nd-opinion has-patch => 2nd-opinion has-patch close
Comment:
Replying to [comment:17 danielbachhuber]:
> I disagree. Creating a `WP_Error` object is a deliberate decision of the
developer to report an error upstream. Returning false is for a valid
condition.
FWIW, I agree with nacin. The REST API will make pretty liberal use of
WP_Error (intentionally), and logging all of those alone would be
problematic. In addition, there's the possibility for a DoS attack using
it if errors are logged to a DB.
Recommend close as wontfix.
--
Ticket URL: <https://core.trac.wordpress.org/ticket/28319#comment:20>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform
More information about the wp-trac
mailing list