Changeset 57882

Timestamp:

03/26/2024 09:39:36 PM (2 years ago)

Author:

SergeyBiryukov

Message:

Coding Standards: Use strict comparison in wp-includes/pluggable.php.

Follow-up to [3566], [6387], [10437], [11057], [11387], [16208], [16304], [18195], [20410], [26367], [34947].

Props aristath, poena, afercia, SergeyBiryukov.
See #60700.

File:

• trunk/src/wp-includes/pluggable.php (modified) (5 diffs)

trunk/src/wp-includes/pluggable.php

@@ -31 +31 @@
                 if ( isset( $current_user )
                 && ( $current_user instanceof WP_User )
-                && ( $id == $current_user->ID )
+                && ( $id === $current_user->ID )
                 && ( null !== $id )
                 ) {
@@ -618 +618 @@
                 $user = apply_filters( 'authenticate', null, $username, $password );
 
-                if ( null == $user ) {
+                if ( null === $user ) {
                         /*
                          * TODO: What should the error message be? (Or would these even happen?)
@@ -1094 +1094 @@
                 setcookie( $auth_cookie_name, $auth_cookie, $expire, ADMIN_COOKIE_PATH, COOKIE_DOMAIN, $secure, true );
                 setcookie( LOGGED_IN_COOKIE, $logged_in_cookie, $expire, COOKIEPATH, COOKIE_DOMAIN, $secure_logged_in_cookie, true );
-                if ( COOKIEPATH != SITECOOKIEPATH ) {
+                if ( COOKIEPATH !== SITECOOKIEPATH ) {
                         setcookie( LOGGED_IN_COOKIE, $logged_in_cookie, $expire, SITECOOKIEPATH, COOKIE_DOMAIN, $secure_logged_in_cookie, true );
                 }
@@ -1316 +1316 @@
          */
         function check_ajax_referer( $action = -1, $query_arg = false, $stop = true ) {
-                if ( -1 == $action ) {
+                if ( -1 === $action ) {
                         _doing_it_wrong( __FUNCTION__, __( 'You should specify an action to be verified by using the first parameter.' ), '4.7.0' );
                 }
@@ -1699 +1699 @@
 
                 // The comment was left by the author.
-                if ( $author && ! $notify_author && $comment->user_id == $post->post_author ) {
+                if ( $author && ! $notify_author && (int) $comment->user_id === (int) $post->post_author ) {
                         unset( $emails[ $author->user_email ] );
                 }
 
                 // The author moderated a comment on their own post.
-                if ( $author && ! $notify_author && get_current_user_id() == $post->post_author ) {
+                if ( $author && ! $notify_author && get_current_user_id() === (int) $post->post_author ) {
                         unset( $emails[ $author->user_email ] );
                 }