Ticket #22192: 22192.3.diff

src/wp-includes/option.php

@@ -264 +264 @@
          */
         $value = apply_filters( 'pre_update_option', $value, $option, $old_value );
 
-        // If the new and old values are the same, no need to update.
-        if ( $value === $old_value )
+        /*
+         * If the new and old values are the same, no need to update.
+         * Scalar values in the cache will always be strings, so we must compare string values.
+         */
+        if ( is_scalar( $value ) && is_scalar( $old_value ) ) {
+                $_value = (string) $value;
+                $_old_value = (string) $old_value;
+        } else {
+                $_value = $value;
+                $_old_value = $old_value;
+        }
+
+        if ( $_value === $_old_value )
                 return false;
 
         if ( false === $old_value )

tests/phpunit/tests/option/option.php

@@ -99 +99 @@
         function test_special_option_name_notoptions() {
                 delete_option( 'notoptions' );
         }
+
+        /**
+         * @ticket 22192
+         * @dataProvider data_update_option_type_juggling
+         */
+        public function test_update_option_should_hit_cache_when_loosely_equal_to_existing_value( $old_value, $new_value ) {
+                global $wpdb;
+
+                add_option( 'foo', $old_value );
+                $num_queries = $wpdb->num_queries;
+
+                $updated = update_option( 'foo', $new_value );
+
+                $this->assertFalse( $updated );
+                $this->assertSame( $num_queries, $wpdb->num_queries );
+        }
+
+        public function data_update_option_type_juggling() {
+                return array(
+                        // Truthy
+                        array( '1', '1' ),
+                        array( '1', intval( 1 ) ),
+                        array( '1', floatval( 1 ) ),
+                        array( '1', true ),
+                        array( 1, '1' ),
+                        array( 1, intval( 1 ) ),
+                        array( 1, floatval( 1 ) ),
+                        array( 1, true ),
+                        array( floatval( 1 ), '1' ),
+                        array( floatval( 1 ), intval( 1 ) ),
+                        array( floatval( 1 ), floatval( 1 ) ),
+                        array( floatval( 1 ), true ),
+                        array( true, '1' ),
+                        array( true, intval( 1 ) ),
+                        array( true, floatval( 1 ) ),
+                        array( true, true ),
+
+                        // Falsey
+                        array( '0', '0' ),
+                        array( '0', intval( 0 ) ),
+                        array( '0', floatval( 0 ) ),
+                        array( '0', false ),
+                        array( 0, '0' ),
+                        array( 0, intval( 0 ) ),
+                        array( 0, floatval( 0 ) ),
+                        array( 0, false ),
+                        array( floatval( 0 ), '0' ),
+                        array( floatval( 0 ), intval( 0 ) ),
+                        array( floatval( 0 ), floatval( 0 ) ),
+                        array( floatval( 0 ), false ),
+                        array( false, '0' ),
+                        array( false, intval( 0 ) ),
+                        array( false, floatval( 0 ) ),
+                        array( false, false ),
+                );
+        }
 }