From 05566e992d764b5eda1752e5b52b1b6d4a6ae143 Mon Sep 17 00:00:00 2001 From: Sergey Biryukov Date: Sun, 21 Mar 2021 12:39:25 +0000 Subject: [PATCH] Coding Standards: Use strict comparison for return type checks in a few functions: * `get_bookmark()` * `get_comment()` * `get_post()` * `get_children()` * `wp_get_recent_posts()` * `wp_get_post_revision()` * `wp_get_nav_menu_items()` Follow-up to [45710] for `get_term()`, [48507] for `wpdb::get_row()` and `wpdb::get_results()`. See #52627. git-svn-id: https://develop.svn.wordpress.org/trunk@50558 602fd350-edb4-49c9-b593-d223f7449a82 --- src/wp-includes/bookmark.php | 6 +++--- src/wp-includes/comment.php | 6 +++--- src/wp-includes/nav-menu.php | 6 ++++-- src/wp-includes/post.php | 12 ++++++------ src/wp-includes/revision.php | 6 +++--- 5 files changed, 19 insertions(+), 17 deletions(-) diff --git a/src/wp-includes/bookmark.php b/src/wp-includes/bookmark.php index b9cfc63765ab..674269e23b73 100644 --- a/src/wp-includes/bookmark.php +++ b/src/wp-includes/bookmark.php @@ -53,11 +53,11 @@ function get_bookmark( $bookmark, $output = OBJECT, $filter = 'raw' ) { $_bookmark = sanitize_bookmark( $_bookmark, $filter ); - if ( OBJECT == $output ) { + if ( OBJECT === $output ) { return $_bookmark; - } elseif ( ARRAY_A == $output ) { + } elseif ( ARRAY_A === $output ) { return get_object_vars( $_bookmark ); - } elseif ( ARRAY_N == $output ) { + } elseif ( ARRAY_N === $output ) { return array_values( get_object_vars( $_bookmark ) ); } else { return $_bookmark; diff --git a/src/wp-includes/comment.php b/src/wp-includes/comment.php index 94f60dfaef77..91672c1a13de 100644 --- a/src/wp-includes/comment.php +++ b/src/wp-includes/comment.php @@ -218,11 +218,11 @@ function get_comment( $comment = null, $output = OBJECT ) { */ $_comment = apply_filters( 'get_comment', $_comment ); - if ( OBJECT == $output ) { + if ( OBJECT === $output ) { return $_comment; - } elseif ( ARRAY_A == $output ) { + } elseif ( ARRAY_A === $output ) { return $_comment->to_array(); - } elseif ( ARRAY_N == $output ) { + } elseif ( ARRAY_N === $output ) { return array_values( $_comment->to_array() ); } return $_comment; diff --git a/src/wp-includes/nav-menu.php b/src/wp-includes/nav-menu.php index 547ca7f709ac..50f43f955dc5 100644 --- a/src/wp-includes/nav-menu.php +++ b/src/wp-includes/nav-menu.php @@ -758,14 +758,16 @@ function wp_get_nav_menu_items( $menu, $args = array() ) { $items = array_filter( $items, '_is_valid_nav_menu_item' ); } - if ( ARRAY_A == $args['output'] ) { + if ( ARRAY_A === $args['output'] ) { $items = wp_list_sort( $items, array( $args['output_key'] => 'ASC', ) ); - $i = 1; + + $i = 1; + foreach ( $items as $k => $item ) { $items[ $k ]->{$args['output_key']} = $i++; } diff --git a/src/wp-includes/post.php b/src/wp-includes/post.php index d543c4176ebc..b17f2546531a 100644 --- a/src/wp-includes/post.php +++ b/src/wp-includes/post.php @@ -677,15 +677,15 @@ function get_children( $args = '', $output = OBJECT ) { $kids[ $child->ID ] = $children[ $key ]; } - if ( OBJECT == $output ) { + if ( OBJECT === $output ) { return $kids; - } elseif ( ARRAY_A == $output ) { + } elseif ( ARRAY_A === $output ) { $weeuns = array(); foreach ( (array) $kids as $kid ) { $weeuns[ $kid->ID ] = get_object_vars( $kids[ $kid->ID ] ); } return $weeuns; - } elseif ( ARRAY_N == $output ) { + } elseif ( ARRAY_N === $output ) { $babes = array(); foreach ( (array) $kids as $kid ) { $babes[ $kid->ID ] = array_values( get_object_vars( $kids[ $kid->ID ] ) ); @@ -788,9 +788,9 @@ function get_post( $post = null, $output = OBJECT, $filter = 'raw' ) { $_post = $_post->filter( $filter ); - if ( ARRAY_A == $output ) { + if ( ARRAY_A === $output ) { return $_post->to_array(); - } elseif ( ARRAY_N == $output ) { + } elseif ( ARRAY_N === $output ) { return array_values( $_post->to_array() ); } @@ -3692,7 +3692,7 @@ function wp_get_recent_posts( $args = array(), $output = ARRAY_A ) { $results = get_posts( $parsed_args ); // Backward compatibility. Prior to 3.1 expected posts to be returned in array. - if ( ARRAY_A == $output ) { + if ( ARRAY_A === $output ) { foreach ( $results as $key => $result ) { $results[ $key ] = get_object_vars( $result ); } diff --git a/src/wp-includes/revision.php b/src/wp-includes/revision.php index 6fc288888de4..a1aad16812ad 100644 --- a/src/wp-includes/revision.php +++ b/src/wp-includes/revision.php @@ -371,12 +371,12 @@ function wp_get_post_revision( &$post, $output = OBJECT, $filter = 'raw' ) { return null; } - if ( OBJECT == $output ) { + if ( OBJECT === $output ) { return $revision; - } elseif ( ARRAY_A == $output ) { + } elseif ( ARRAY_A === $output ) { $_revision = get_object_vars( $revision ); return $_revision; - } elseif ( ARRAY_N == $output ) { + } elseif ( ARRAY_N === $output ) { $_revision = array_values( get_object_vars( $revision ) ); return $_revision; }