Changeset 55642

Timestamp:

04/10/2023 12:54:28 PM (2 years ago)

Author:

SergeyBiryukov

Message:

Coding Standards: Use strict comparison where count() is involved.

Follow-up to [1636], [6974], [8114], [10322], [13326], [14760], [18006], [18541], [19743], [23249], [24115], [33359].

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

Location:

trunk/src

Files:

• wp-admin/includes/class-wp-importer.php (modified) (2 diffs)
• wp-admin/includes/menu.php (modified) (1 diff)
• wp-includes/class-wp-query.php (modified) (1 diff)
• wp-includes/class-wp-rewrite.php (modified) (1 diff)
• wp-includes/class-wp-xmlrpc-server.php (modified) (1 diff)
• wp-includes/deprecated.php (modified) (1 diff)
• wp-includes/formatting.php (modified) (5 diffs)
• wp-includes/functions.php (modified) (1 diff)
• wp-includes/kses.php (modified) (1 diff)
• wp-includes/ms-functions.php (modified) (1 diff)
• wp-includes/pluggable.php (modified) (1 diff)
• wp-includes/post.php (modified) (1 diff)
• wp-includes/template.php (modified) (1 diff)

trunk/src/wp-admin/includes/class-wp-importer.php

@@ -42 +42 @@
                 }
             }
-        } while ( count( $results ) == $limit );
+        } while ( count( $results ) === $limit );
 
         return $hashtable;
@@ -111 +111 @@
                 }
             }
-        } while ( count( $results ) == $limit );
+        } while ( count( $results ) === $limit );
 
         return $hashtable;

trunk/src/wp-admin/includes/menu.php

@@ -242 +242 @@
         }
 
-        if ( $i == $items_count ) { // Last item.
+        if ( $i === $items_count ) { // Last item.
             $classes           = $menu[ $order ][4];
             $menu[ $order ][4] = add_cssclass( 'menu-top-last', $classes );

trunk/src/wp-includes/class-wp-query.php

@@ -2253 +2253 @@
                 if ( ! $post_type ) {
                     $post_type = 'any';
-                } elseif ( count( $post_type ) == 1 ) {
+                } elseif ( count( $post_type ) === 1 ) {
                     $post_type = $post_type[0];
                 }

trunk/src/wp-includes/class-wp-rewrite.php

@@ -1400 +1400 @@
         foreach ( $this->extra_permastructs as $permastructname => $struct ) {
             if ( is_array( $struct ) ) {
-                if ( count( $struct ) == 2 ) {
+                if ( count( $struct ) === 2 ) {
                     $rules = $this->generate_rewrite_rules( $struct[0], $struct[1] );
                 } else {

trunk/src/wp-includes/class-wp-xmlrpc-server.php

@@ -4244 +4244 @@
 
         // If no specific options where asked for, return all of them.
-        if ( count( $options ) == 0 ) {
+        if ( count( $options ) === 0 ) {
             $options = array_keys( $this->blog_options );
         }

trunk/src/wp-includes/deprecated.php

@@ -2298 +2298 @@
     $metas = array_map('maybe_unserialize', $metas);
 
-    if ( count($metas) == 1 )
+    if ( count($metas) === 1 )
         return $metas[0];
     else

trunk/src/wp-includes/formatting.php

@@ -1170 +1170 @@
             $unicode_length += $encoded_char_length;
         } else {
-            if ( count( $values ) == 0 ) {
+            if ( count( $values ) === 0 ) {
                 if ( $value < 224 ) {
                     $num_octets = 2;
@@ -1185 +1185 @@
                 break;
             }
-            if ( count( $values ) == $num_octets ) {
+            if ( count( $values ) === $num_octets ) {
                 for ( $j = 0; $j < $num_octets; $j++ ) {
                     $unicode .= '%' . dechex( $values[ $j ] );
@@ -3392 +3392 @@
     global $wpsmiliestrans;
 
-    if ( count( $matches ) == 0 ) {
+    if ( count( $matches ) === 0 ) {
         return '';
     }
@@ -5243 +5243 @@
     $args   = (array) $args;
     $result = array_shift( $args );
-    if ( count( $args ) == 1 ) {
+    if ( count( $args ) === 1 ) {
         $result .= $l['between_only_two'] . array_shift( $args );
     }
@@ -5252 +5252 @@
         $arg = array_shift( $args );
         $i--;
-        if ( 0 == $i ) {
+        if ( 0 === $i ) {
             $result .= $l['between_last_two'] . $arg;
         } else {

trunk/src/wp-includes/functions.php

@@ -4755 +4755 @@
     $wpsmiliestrans = apply_filters( 'smilies', $wpsmiliestrans );
 
-    if ( count( $wpsmiliestrans ) == 0 ) {
+    if ( count( $wpsmiliestrans ) === 0 ) {
         return;
     }

trunk/src/wp-includes/kses.php

@@ -789 +789 @@
     $split = preg_split( '/\s*=\s*/', $attr, 2 );
     $name  = $split[0];
-    if ( count( $split ) == 2 ) {
+    if ( count( $split ) === 2 ) {
         $value = $split[1];
 

trunk/src/wp-includes/ms-functions.php

@@ -275 +275 @@
 
     $blogs = get_blogs_of_user( $user_id );
-    if ( count( $blogs ) == 0 ) {
+    if ( count( $blogs ) === 0 ) {
         update_user_meta( $user_id, 'primary_blog', '' );
         update_user_meta( $user_id, 'source_domain', '' );

trunk/src/wp-includes/pluggable.php

@@ -433 +433 @@
 
                     if ( preg_match( '/(.*)<(.+)>/', $address, $matches ) ) {
-                        if ( count( $matches ) == 3 ) {
+                        if ( count( $matches ) === 3 ) {
                             $recipient_name = $matches[1];
                             $address        = $matches[2];

trunk/src/wp-includes/post.php

@@ -5733 +5733 @@
             }
 
-            if ( 0 == $p->post_parent && count( $revparts ) == $count + 1 && $p->post_name == $revparts[ $count ] ) {
+            if ( 0 == $p->post_parent && count( $revparts ) === $count + 1 && $p->post_name == $revparts[ $count ] ) {
                 $foundid = $page->ID;
                 if ( $page->post_type == $post_type ) {

trunk/src/wp-includes/template.php

@@ -153 +153 @@
     $templates = array();
 
-    if ( count( $post_types ) == 1 ) {
+    if ( count( $post_types ) === 1 ) {
         $post_type   = reset( $post_types );
         $templates[] = "archive-{$post_type}.php";