Ticket #42069: 42069.5.diff

src/wp-includes/rest-api/fields/class-wp-rest-meta-fields.php

@@ -322 +322 @@
                 $old_value = get_metadata( $meta_type, $object_id, $meta_key );
 
                 if ( 1 === count( $old_value ) ) {
-                        if ( $old_value[0] === $meta_value ) {
+                        if ( (string) sanitize_meta( $meta_key, wp_unslash( $meta_value ), $meta_type ) === $old_value[0] ) {
                                 return true;
                         }
                 }

tests/phpunit/tests/rest-api/rest-post-meta-fields.php

@@ -130 +130 @@
                         )
                 );
 
+                register_meta(
+                        'post', 'test_boolean_update', array(
+                                'single'            => true,
+                                'type'              => 'boolean',
+                                'sanitize_callback' => 'absint',
+                                'show_in_rest'      => true,
+                        )
+                );
+                register_meta(
+                        'post', 'test_textured_text_update', array(
+                                'single'            => true,
+                                'type'              => 'string',
+                                'sanitize_callback' => 'sanitize_text_field',
+                                'show_in_rest'      => true,
+                        )
+                );
+
+                register_meta(
+                        'post', 'test_json_text_update', array(
+                                'single'            => true,
+                                'type'              => 'string',
+                                'show_in_rest'      => true,
+                        )
+                );
+
                 register_post_type( 'cpt', array(
                         'show_in_rest' => true,
                         'supports'     => array( 'custom-fields' ),
@@ -1199 +1224 @@
                 return $data;
         }
 
+        /**
+         * @ticket 42069
+         * @dataProvider data_update_value_return_success_with_same_value
+         */
+        public function test_update_value_return_success_with_same_value( $meta_key, $meta_value ) {
+                add_post_meta( self::$post_id, $meta_key, $meta_value );
+
+                $this->grant_write_permission();
+
+                $data = array(
+                        'meta' => array(
+                                $meta_key => $meta_value,
+                        ),
+                );
+
+                $request = new WP_REST_Request( 'POST', sprintf( '/wp/v2/posts/%d', self::$post_id ) );
+                $request->set_body_params( $data );
+
+                $response = rest_get_server()->dispatch( $request );
+
+                $this->assertEquals( 200, $response->get_status() );
+        }
+
+        public function data_update_value_return_success_with_same_value() {
+                return array(
+                        array( 'test_boolean_update', 0 ),
+                        array( 'test_boolean_update', 1 ),
+                        array( 'test_boolean_update', false ),
+                        array( 'test_boolean_update', true ),
+                        array( 'test_boolean_update', '' ),
+                        array( 'test_boolean_update', '0' ),
+                        array( 'test_boolean_update', '1' ),
+                        array( 'test_textured_text_update', 'She said, "What about the > 10,000 penguins in the kitchen?"' ),
+                        array( 'test_textured_text_update', "He's about to do something rash..." ),
+                        array( 'test_json_text_update', json_encode( array( 'foo' => 'bar' ) ) ),
+                );
+        }
+
+        /**
+         * @ticket 42069
+         * @dataProvider data_update_value_return_success_with_similar_value
+         */
+        public function test_update_value_return_success_with_similar_value( $meta_key, $meta_value, $update_meta_value ) {
+                add_post_meta( self::$post_id, $meta_key, $meta_value );
+
+                $this->grant_write_permission();
+
+                $data = array(
+                        'meta' => array(
+                                $meta_key => $update_meta_value,
+                        ),
+                );
+
+                $request = new WP_REST_Request( 'POST', sprintf( '/wp/v2/posts/%d', self::$post_id ) );
+                $request->set_body_params( $data );
+
+                $response = rest_get_server()->dispatch( $request );
+
+                $this->assertEquals( 200, $response->get_status() );
+        }
+
+        public function data_update_value_return_success_with_similar_value() {
+                return array(
+                        array( 'test_boolean_update', 0, '' ),
+                        array( 'test_boolean_update', 1, '1' ),
+                        array( 'test_boolean_update', false, 0 ),
+                        array( 'test_boolean_update', true, 1 ),
+                        array( 'test_boolean_update', '', false ),
+                        array( 'test_boolean_update', '1', true ),
+                );
+        }
+
         /**
          * Internal function used to disable an insert query which
          * will trigger a wpdb error for testing purposes.