[wp-trac] [WordPress Trac] #28319: Add hook to WP_Error for logging and report purposes
WordPress Trac
noreply at wordpress.org
Mon Nov 24 15:28:37 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: 4.0
Severity: normal | Resolution:
Keywords: 2nd-opinion has-patch | Focuses:
-----------------------------------+-----------------------------
Comment (by sboisvert):
>Then WordPress needs to add a is_wp_check() to the result of
get_terms_to_edit() and it's friends before attempting to use it as a
string then. I've created #30472 for that.
That was one example and while that patch will solve the fatal it won't
address the underlying issue. It will just make it fail silently instead
of having it work as expected. I think the overall idea of being able to
hook into WP_Errors is still very important.
>My reasoning was unclear here; I meant if we ever added actual logging
for this by default.
I agree it would be far too much noise. I think the only time this would
be done is in a debugging plugin (or potentially with WP_DEBUG enabled in
a dev environment).
--
Ticket URL: <https://core.trac.wordpress.org/ticket/28319#comment:27>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform
More information about the wp-trac
mailing list