Remove @static tags from core DocBlocks

[birgire]

This ticket is for the removal of the @static tag from core docblocks but not the @staticvar tag for static variables.

The @static tag is redundant and similar has been done regarding the @access tag in #41452.

Also see the comment by @DrewAPicture in ticket:41401#comment:2

The PHPDocumentator ​mentions that @static was a helpful tag in PHP-4 but isn't needed for static keyword usage in PHP-5.

[birgire]

The wp-admin/* classes

[birgire]

The wp-includes/* and wp-content/* classes

[birgire]

42803-wp-admin.diff​ is for the relevant .php files in wp-admin/*.

42803-wp-includes-wp-content.diff​ is for the relevant .php files in wp-includes/* and wp-content/*. It skips these vendor files:

• wp-includes/class-phpmailer.php
• wp-includes/Text/Diff.php
• wp-includes/SimplePie/Misc.php

Hopefully this covers it.

[netweb]

Looks good to me

[SergeyBiryukov]

In 42745:

Docs: Remove @static notations from method DocBlocks in wp-admin/* classes.

This tag has been used in the past, but should no longer be used. Just using the static keyword in code is enough for PhpDocumentor on PHP5+ to recognize static variables and methods, and PhpDocumentor will mark them as static.

Props birgire.
See #42803.

[SergeyBiryukov]

In 42746:

Docs: Remove @static notations from method DocBlocks in wp-includes/* classes.

This tag has been used in the past, but should no longer be used. Just using the static keyword in code is enough for PhpDocumentor on PHP5+ to recognize static variables and methods, and PhpDocumentor will mark them as static.

Props birgire.
See #42803.

[SergeyBiryukov]

In 42747:

Docs: Remove @static notations from property DocBlocks in wp-admin/* and wp-includes/* classes.

This tag has been used in the past, but should no longer be used. Just using the static keyword in code is enough for PhpDocumentor on PHP5+ to recognize static variables and methods, and PhpDocumentor will mark them as static.

Props birgire.
See #42803.

[SergeyBiryukov]

In 42749:

Twenty Fourteen: Remove redundant @access and @static notations from classes.

Props birgire.
See #41452, #42803, #42505.

[SergeyBiryukov]

In 42751:

Docs: Remove erroneous @static notation from WP_Http::_dispatch_request(), added in [32650].

The method was never user as static.

Props birgire.
See #42803.

[SergeyBiryukov]

In 42752:

Docs: Remove erroneous @static notation from WP_Tax_Query::get_sql(), added in [32627].

The method was never used as static.

Props birgire.
See #42803.

[SergeyBiryukov]

The only instance left is MO::make_entry() in wp-includes/pomo/mo.php.

Looks like it was never used as static in core, though could potentially be called as static. MO::get_byteorder(), on the other hand, is used as static, but is not marked as such.

The whole POMO library does not have access modifiers (it was skipped in #22234, see comment:16:ticket:22234). We could probably reconsider it, but let's do that in a new ticket.

[SergeyBiryukov]

Follow-up: #43424