Scrutinizer in 4.7

[wonderboymusic]

Remove unused code, etc. It has been a long time since I have run these scans.

[wonderboymusic]

In 38308:

General: remove variable set needlessly in wp_check_jsonp_callback().

See #37771.

[wonderboymusic]

In 38309:

Media: remove unnecessary variable assignment in gallery_shortcode().

See #37771.

[wonderboymusic]

In 38310:

REST API: remove unnecessary variable assignments in rest_handle_options_request().

See #37771.

[wonderboymusic]

In 38311:

Nav Menus: remove unnecessary variable assignment in wp_nav_menu_item_post_type_meta_box().

See #37771.

[wonderboymusic]

In 38312:

Taxonomy: remove unnecessary break in WP_Term::__get().

See #37771.

[wonderboymusic]

In 38313:

Comments: in wp_handle_comment_submission(), $_wp_unfiltered_html_comment is passed as part of $comment_data, but is not used locally.

See #37771.

[wonderboymusic]

In 38314:

Database: WP_Network, WP_Network_Query, and WP_Site_Query call wpdb::_escape(), thus requiring it to be public. It previously had no access modifier. _ at the beginning of a method, believe it or not, does not enforce visibility constraints.

See #37771.

[wonderboymusic]

In 38315:

HTTP: in WP_HTTP_Response, the @param declarations for $status and $headers were swapped. Let us correct this.

See #37771.

[wonderboymusic]

In 38316:

Locale: declare the $month_genitive field on WP_Locale.

See #37771.

[wonderboymusic]

In 38317:

Users: $user_level has been publicly-accessed on instances of WP_User since version 2.0, but is has never been declared.

See #37771.

[wonderboymusic]

In 38318:

Widgets: $option_name and $alt_option_name have been used as members ever since WP_Widget became an object in 2.8, but never declared.

See #37771.

[wonderboymusic]

In 38319:

Users: after [38317], use a @property annotation, instead of a public field.

See #37771.

[jorbin]

@wonderboymusic is there more that you plan to do here for 4.7, or can this be closed out?

[helen]

No further movement, closing as fixed.