Introduce _doing_it_wrong to mark places where plugins / themes are calling functions incorrectly

[westi]

We are starting to mark incorrect usage of functions with _deprecated_function where as we should have a specific way of marking these so the message is correct as we are not deprecating something but telling people the code they have written won't work

[westi]

Example: Notice: register_uninstall_hook was called with an argument that is deprecated since version 3.1! Only a static class method or function can be used in an uninstall hook.

[nacin]

That's the only one I can really think of so far.

In the pipeline, there's #14729 and #11526. #14729 is still on the 3.1 milestone and that one sounds good.

I'll be incorporating this into my Log Deprecated Notices plugin. Please make it easy to backtrace ;-)

[westi]

(In [16939]) Introduce _doing_it_wrong(). See #15824.

[westi]

(In [16940]) Switch to _doing_it_wrong(). See #15824

[westi]

Not sure if there are any other places we are using _deprecated_argument when we should be using this.

[nacin]

Just checked them all. We could really use a _deprecated_message() function for a few cases where we've overloaded _deprecated_argument(), but all are indeed deprecated instances of arguments or something.

[westi]

(In [16942]) Better message text to reflect that it might not just be an incorrect argument but maybe your calling something way too early. See #15824

[nacin]

(In [16945]) Move _doing_it_wrong to other debug functions. Modify docs and the output message. see #15824.

[nacin]

(In [16946]) Grammar and spelling. see #15824.

[nacin]

(In [16947]) You are doing it wrong in query.php. see #15824, see #14729.

[ChristianKopp]

sorry for reopen this ticket.

getting this message in the dashboard after updating to the latest trunk (from yesterday)

Warning: Missing argument 3 for _doing_it_wrong(), called in /wp-includes/plugin.php on line 663 and defined in /wp-includes/functions.php on line 3404

have you all a great xmas time

[nacin]

(In [16955]) _doing_it_wrong takes three arguments. fixes #15824.

[nacin]

You should figure out which plugin is doing it wrong, because one of them is :-)

It'll look like this:

register_uninstall_hook( __FILE__, array( &$this, 'some_method' );

[filosofo]

Great. Now you're going to have to make a _doing_doing_it_wrong_wrong function.

[WraithKenny]

related #18263