[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