WordPress.org

Make WordPress Core


Ignore:
Timestamp:
05/16/2020 06:40:52 PM (11 months 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.