WordPress.org

Make WordPress Core


Ignore:
Timestamp:
04/05/2020 03:00:44 AM (12 months ago)
Author:
SergeyBiryukov
Message:

Coding Standards: Use strict type check for in_array() and array_search() where strings are involved.

This reduces the number of WordPress.PHP.StrictInArray.MissingTrueStrict issues from 486 to 50.

Includes minor code layout fixes for better readability.

See #49542.

File:
1 edited

Legend:

Unmodified
Added
Removed
  • trunk/src/wp-includes/class-wp-xmlrpc-server.php

    r47233 r47550  
    758758        );
    759759
    760         if ( in_array( 'labels', $fields ) ) {
     760        if ( in_array( 'labels', $fields, true ) ) {
    761761            $_taxonomy['labels'] = (array) $taxonomy->labels;
    762762        }
    763763
    764         if ( in_array( 'cap', $fields ) ) {
     764        if ( in_array( 'cap', $fields, true ) ) {
    765765            $_taxonomy['cap'] = (array) $taxonomy->cap;
    766766        }
    767767
    768         if ( in_array( 'menu', $fields ) ) {
     768        if ( in_array( 'menu', $fields, true ) ) {
    769769            $_taxonomy['show_in_menu'] = (bool) $_taxonomy->show_in_menu;
    770770        }
    771771
    772         if ( in_array( 'object_type', $fields ) ) {
     772        if ( in_array( 'object_type', $fields, true ) ) {
    773773            $_taxonomy['object_type'] = array_unique( (array) $taxonomy->object_type );
    774774        }
     
    903903
    904904        // Merge requested $post_fields fields into $_post.
    905         if ( in_array( 'post', $fields ) ) {
     905        if ( in_array( 'post', $fields, true ) ) {
    906906            $_post = array_merge( $_post, $post_fields );
    907907        } else {
     
    910910        }
    911911
    912         $all_taxonomy_fields = in_array( 'taxonomies', $fields );
    913 
    914         if ( $all_taxonomy_fields || in_array( 'terms', $fields ) ) {
     912        $all_taxonomy_fields = in_array( 'taxonomies', $fields, true );
     913
     914        if ( $all_taxonomy_fields || in_array( 'terms', $fields, true ) ) {
    915915            $post_type_taxonomies = get_object_taxonomies( $post['post_type'], 'names' );
    916916            $terms                = wp_get_object_terms( $post['ID'], $post_type_taxonomies );
     
    921921        }
    922922
    923         if ( in_array( 'custom_fields', $fields ) ) {
     923        if ( in_array( 'custom_fields', $fields, true ) ) {
    924924            $_post['custom_fields'] = $this->get_custom_fields( $post['ID'] );
    925925        }
    926926
    927         if ( in_array( 'enclosure', $fields ) ) {
     927        if ( in_array( 'enclosure', $fields, true ) ) {
    928928            $_post['enclosure'] = array();
    929929            $enclosures         = (array) get_post_meta( $post['ID'], 'enclosure' );
     
    970970        );
    971971
    972         if ( in_array( 'labels', $fields ) ) {
     972        if ( in_array( 'labels', $fields, true ) ) {
    973973            $_post_type['labels'] = (array) $post_type->labels;
    974974        }
    975975
    976         if ( in_array( 'cap', $fields ) ) {
     976        if ( in_array( 'cap', $fields, true ) ) {
    977977            $_post_type['cap']          = (array) $post_type->cap;
    978978            $_post_type['map_meta_cap'] = (bool) $post_type->map_meta_cap;
    979979        }
    980980
    981         if ( in_array( 'menu', $fields ) ) {
     981        if ( in_array( 'menu', $fields, true ) ) {
    982982            $_post_type['menu_position'] = (int) $post_type->menu_position;
    983983            $_post_type['menu_icon']     = $post_type->menu_icon;
     
    985985        }
    986986
    987         if ( in_array( 'taxonomies', $fields ) ) {
     987        if ( in_array( 'taxonomies', $fields, true ) ) {
    988988            $_post_type['taxonomies'] = get_object_taxonomies( $post_type->name, 'names' );
    989989        }
     
    11911191        );
    11921192
    1193         if ( in_array( 'all', $fields ) ) {
     1193        if ( in_array( 'all', $fields, true ) ) {
    11941194            $_user = array_merge( $_user, $user_fields );
    11951195        } else {
    1196             if ( in_array( 'basic', $fields ) ) {
     1196            if ( in_array( 'basic', $fields, true ) ) {
    11971197                $basic_fields = array( 'username', 'email', 'registered', 'display_name', 'nicename' );
    11981198                $fields       = array_merge( $fields, $basic_fields );
     
    15781578                    $term_names = $post_data['terms_names'][ $taxonomy ];
    15791579                    foreach ( $term_names as $term_name ) {
    1580                         if ( in_array( $term_name, $ambiguous_terms ) ) {
     1580                        if ( in_array( $term_name, $ambiguous_terms, true ) ) {
    15811581                            return new IXR_Error( 401, __( 'Ambiguous term name used in a hierarchical taxonomy. Please use term ID instead.' ) );
    15821582                        }
     
    37583758            $statuses = array_keys( $statuses );
    37593759
    3760             if ( ! in_array( $content_struct['status'], $statuses ) ) {
     3760            if ( ! in_array( $content_struct['status'], $statuses, true ) ) {
    37613761                return new IXR_Error( 401, __( 'Invalid comment status.' ) );
    37623762            }
     
    56695669
    56705670        // Use wp.editPost to edit post types other than post and page.
    5671         if ( ! in_array( $postdata['post_type'], array( 'post', 'page' ) ) ) {
     5671        if ( ! in_array( $postdata['post_type'], array( 'post', 'page' ), true ) ) {
    56725672            return new IXR_Error( 401, __( 'Invalid post type.' ) );
    56735673        }
Note: See TracChangeset for help on using the changeset viewer.