Scrutinizer in 4.4

[wonderboymusic]

Static analysis, uncovers a lot of edge cases

[wonderboymusic]

In 33697:

In wp_ajax_add_meta(), do not juggle the value of $_POST and alter it directly. This was done so that edit_post() could pull $_POST out of the air by-reference and alter it (equally as bad). edit_post() accepts a $post_data array. Do that instead.

See #33491.

[wonderboymusic]

In 33699:

In _WP_Editors::editor_settings(), $first_run is unused.

See #33491.

[wonderboymusic]

In 33700:

Doc block for_wp_specialchars: $quote_style can also be string ('single' or 'double')

See #33491.

[wonderboymusic]

In 33701:

In wpdb::get_col_length(), breaks are not necessary when a case returns

See #33491.

[wonderboymusic]

In 33734:

foreach is a statement, not a function.

See #33491.

[wonderboymusic]

In 33735:

In WP_Users_List_Table::single_row(), $actions is not always set before being used.

See #33491.

[wonderboymusic]

In 33956:

In wpdb::get_col_length(), the final return false is unreachable since the default switch case returns.

See #33491.

[wonderboymusic]

In 33957:

WP_Theme has an ad hoc property in WP_MS_Themes_List_Table, $update. This can be set to a default value on the class, as it's not obtained via __get().

See #33491.

[wonderboymusic]

In 33958:

In WP_User, add @property docs for description, first_name, and last_name. user_firstname and user_lastname only exist for back-compat.

See #33491.

[DrewAPicture]

In 34236:

Docs: Fix some syntactical issues with the DocBlock for the WP_Theme->update property, introduced in [33957].

Adds an @since.

See #33491. See #32246.

[wonderboymusic]

In 34446:

Remove some unused globals and/or their docs.

See ##33491.

[wonderboymusic]

In 35013:

Docs: some @global object vernaculars should be converted to the actual object type.

See #33491.