[wp-trac] [WordPress Trac] #34183: Add context to `doing_it_wrong_trigger_error` filter

WordPress Trac noreply at wordpress.org
Sun Nov 15 10:44:00 UTC 2015


#34183: Add context to `doing_it_wrong_trigger_error` filter
-----------------------------------+------------------------------
 Reporter:  dd32                   |       Owner:
     Type:  defect (bug)           |      Status:  new
 Priority:  normal                 |   Milestone:  Awaiting Review
Component:  General                |     Version:
 Severity:  normal                 |  Resolution:
 Keywords:  2nd-opinion has-patch  |     Focuses:
-----------------------------------+------------------------------
Changes (by flixos90):

 * keywords:  needs-patch => 2nd-opinion has-patch


Comment:

 In the above patch, I added all the three arguments to the function. It's
 probably most usable that way, at least we need `$function` and `$message`
 I think. We could remove `$version` if it is too specific, but I'd say it
 can be in there.

 In addition to that, I'm proposing to do the same change for the
 `deprecated_argument_trigger_error` filter. Whenever we agree on which
 parameters exactly should be added to the filter (whether it's like in the
 patch above or not), we can simply apply the change to the other filter as
 well.

--
Ticket URL: <https://core.trac.wordpress.org/ticket/34183#comment:2>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform


More information about the wp-trac mailing list