Changeset 56137 for trunk/src/wp-includes/class-wp-list-util.php

Timestamp:

07/05/2023 10:44:20 AM (3 years ago)

Author:

SergeyBiryukov

Message:

General: Compare values as strings in WP_List_Util::filter() and ::sort_callback().

This aims to preserve backward compatibility for code relying on type juggling when using the wp_list_filter() function, e.g. comparing a numeric string to an integer.

Follow-up to [55908].

Props azaozz, jeremyfelt, david.binda.
See #57839.

File:

• trunk/src/wp-includes/class-wp-list-util.php (modified) (2 diffs)

trunk/src/wp-includes/class-wp-list-util.php

@@ -117 +117 @@
                                 if ( is_array( $obj ) ) {
                                         // Treat object as an array.
-                                        if ( array_key_exists( $m_key, $obj ) && ( $m_value === $obj[ $m_key ] ) ) {
+                                        if ( array_key_exists( $m_key, $obj )
+                                                && ( (string) $m_value === (string) $obj[ $m_key ] )
+                                        ) {
                                                 $matched++;
                                         }
                                 } elseif ( is_object( $obj ) ) {
                                         // Treat object as an object.
-                                        if ( isset( $obj->{$m_key} ) && ( $m_value === $obj->{$m_key} ) ) {
+                                        if ( isset( $obj->{$m_key} )
+                                                && ( (string) $m_value === (string) $obj->{$m_key} )
+                                        ) {
                                                 $matched++;
                                         }
@@ -277 +281 @@
                         }
 
-                        if ( $a[ $field ] === $b[ $field ] ) {
+                        if ( (string) $a[ $field ] === (string) $b[ $field ] ) {
                                 continue;
                         }