WordPress.org
Get WordPress

Make WordPress Core


Ignore:
Timestamp:
05/16/2020 06:40:52 PM (6 years ago)
Author:
SergeyBiryukov
Message:

Coding Standards: Use strict comparison where static strings are involved.

This reduces the number of WordPress.PHP.StrictComparisons.LooseComparison issues in half, from 1897 to 890.

Includes minor code layout fixes for better readability.

See #49542.

File:
1 edited

Legend:

Unmodified
Added
Removed

  • trunk/src/wp-includes/nav-menu.php

    r47458 r47808  
    7171                ! is_wp_error( $menu_obj ) &&
    7272                ! empty( $menu_obj->taxonomy ) &&
    73                 'nav_menu' == $menu_obj->taxonomy
     73                'nav_menu' === $menu_obj->taxonomy
    7474        ) {
    7575                return true;
    … …  
    236236 */
    237237function is_nav_menu_item( $menu_item_id = 0 ) {
    238         return ( ! is_wp_error( $menu_item_id ) && ( 'nav_menu_item' == get_post_type( $menu_item_id ) ) );
     238        return ( ! is_wp_error( $menu_item_id ) && ( 'nav_menu_item' === get_post_type( $menu_item_id ) ) );
    239239}
    240240
    … …  
    477477
    478478                $original_title = '';
    479                 if ( 'taxonomy' == $args['menu-item-type'] ) {
     479                if ( 'taxonomy' === $args['menu-item-type'] ) {
    480480                        $original_parent = get_term_field( 'parent', $args['menu-item-object-id'], $args['menu-item-object'], 'raw' );
    481481                        $original_title  = get_term_field( 'name', $args['menu-item-object-id'], $args['menu-item-object'], 'raw' );
    482                 } elseif ( 'post_type' == $args['menu-item-type'] ) {
     482                } elseif ( 'post_type' === $args['menu-item-type'] ) {
    483483
    484484                        $original_object = get_post( $args['menu-item-object-id'] );
    485485                        $original_parent = (int) $original_object->post_parent;
    486486                        $original_title  = $original_object->post_title;
    487                 } elseif ( 'post_type_archive' == $args['menu-item-type'] ) {
     487                } elseif ( 'post_type_archive' === $args['menu-item-type'] ) {
    488488                        $original_object = get_post_type_object( $args['menu-item-object'] );
    489489                        if ( $original_object ) {
    … …  
    497497
    498498                // Hack to get wp to create a post object when too many properties are empty.
    499                 if ( '' == $args['menu-item-title'] && '' == $args['menu-item-description'] ) {
     499                if ( '' === $args['menu-item-title'] && '' === $args['menu-item-description'] ) {
    500500                        $args['menu-item-description'] = ' ';
    501501                }
    … …  
    518518        if ( ! $update ) {
    519519                $post['ID']          = 0;
    520                 $post['post_status'] = 'publish' == $args['menu-item-status'] ? 'publish' : 'draft';
     520                $post['post_status'] = 'publish' === $args['menu-item-status'] ? 'publish' : 'draft';
    521521                $menu_item_db_id     = wp_insert_post( $post );
    522522                if ( ! $menu_item_db_id || is_wp_error( $menu_item_db_id ) ) {
    … …  
    544544        }
    545545
    546         if ( 'custom' == $args['menu-item-type'] ) {
     546        if ( 'custom' === $args['menu-item-type'] ) {
    547547                $args['menu-item-object-id'] = $menu_item_db_id;
    548548                $args['menu-item-object']    = 'custom';
    … …  
    572572        if ( $update ) {
    573573                $post['ID']          = $menu_item_db_id;
    574                 $post['post_status'] = 'draft' == $args['menu-item-status'] ? 'draft' : 'publish';
     574                $post['post_status'] = ( 'draft' === $args['menu-item-status'] ) ? 'draft' : 'publish';
    575575                wp_update_post( $post );
    576576        }
    … …  
    712712                        $type      = get_post_meta( $item->ID, '_menu_item_type', true );
    713713
    714                         if ( 'post_type' == $type ) {
     714                        if ( 'post_type' === $type ) {
    715715                                $posts[ $object ][] = $object_id;
    716                         } elseif ( 'taxonomy' == $type ) {
     716                        } elseif ( 'taxonomy' === $type ) {
    717717                                $terms[ $object ][] = $object_id;
    718718                        }
    … …  
    807807function wp_setup_nav_menu_item( $menu_item ) {
    808808        if ( isset( $menu_item->post_type ) ) {
    809                 if ( 'nav_menu_item' == $menu_item->post_type ) {
     809                if ( 'nav_menu_item' === $menu_item->post_type ) {
    810810                        $menu_item->db_id            = (int) $menu_item->ID;
    811811                        $menu_item->menu_item_parent = ! isset( $menu_item->menu_item_parent ) ? get_post_meta( $menu_item->ID, '_menu_item_menu_item_parent', true ) : $menu_item->menu_item_parent;
    … …  
    814814                        $menu_item->type             = ! isset( $menu_item->type ) ? get_post_meta( $menu_item->ID, '_menu_item_type', true ) : $menu_item->type;
    815815
    816                         if ( 'post_type' == $menu_item->type ) {
     816                        if ( 'post_type' === $menu_item->type ) {
    817817                                $object = get_post_type_object( $menu_item->object );
    818818                                if ( $object ) {
    … …  
    854854                                $menu_item->title = ( '' === $menu_item->post_title ) ? $original_title : $menu_item->post_title;
    855855
    856                         } elseif ( 'post_type_archive' == $menu_item->type ) {
     856                        } elseif ( 'post_type_archive' === $menu_item->type ) {
    857857                                $object = get_post_type_object( $menu_item->object );
    858858                                if ( $object ) {
    … …  
    869869                                $menu_item->url        = get_post_type_archive_link( $menu_item->object );
    870870
    871                         } elseif ( 'taxonomy' == $menu_item->type ) {
     871                        } elseif ( 'taxonomy' === $menu_item->type ) {
    872872                                $object = get_taxonomy( $menu_item->object );
    873873                                if ( $object ) {
    … …  
    10141014                        $menu_item_type = get_post_meta( $menu_item->ID, '_menu_item_type', true );
    10151015                        if (
    1016                                 'post_type' == $object_type &&
    1017                                 'post_type' == $menu_item_type
     1016                                'post_type' === $object_type &&
     1017                                'post_type' === $menu_item_type
    10181018                        ) {
    10191019                                $menu_item_ids[] = (int) $menu_item->ID;
    10201020                        } elseif (
    1021                                 'taxonomy' == $object_type &&
    1022                                 'taxonomy' == $menu_item_type &&
     1021                                'taxonomy' === $object_type &&
     1022                                'taxonomy' === $menu_item_type &&
    10231023                                get_post_meta( $menu_item->ID, '_menu_item_object', true ) == $taxonomy
    10241024                        ) {
    … …  
    10801080 */
    10811081function _wp_auto_add_pages_to_menu( $new_status, $old_status, $post ) {
    1082         if ( 'publish' != $new_status || 'publish' == $old_status || 'page' != $post->post_type ) {
     1082        if ( 'publish' !== $new_status || 'publish' === $old_status || 'page' !== $post->post_type ) {
    10831083                return;
    10841084        }
Note: See TracChangeset for help on using the changeset viewer.