Changeset 55102 for trunk/tests/phpunit/tests/rest-api/rest-posts-controller.php

Timestamp:

01/19/2023 09:46:58 PM (4 years ago)

Author:

SergeyBiryukov

Message:

Tests: Correct additional_field_get_callback() parameters in some REST API tests.

The second parameter passed to get_callback in WP_REST_Controller::add_additional_fields_to_object() is the field name, not the request details.

Includes moving the get_callback and update_callback helper functions next to the tests they are used in.

Follow-up to [38832], [43768].

See #56793.

File:

• trunk/tests/phpunit/tests/rest-api/rest-posts-controller.php (modified) (1 diff)

trunk/tests/phpunit/tests/rest-api/rest-posts-controller.php

@@ -4587 +4587 @@
         }
 
-        public function additional_field_get_callback( $object ) {
-                return get_post_meta( $object['id'], 'my_custom_int', true );
-        }
-
-        public function additional_field_update_callback( $value, $post ) {
+        public function additional_field_get_callback( $object, $field_name ) {
+                return get_post_meta( $object['id'], $field_name, true );
+        }
+
+        public function additional_field_update_callback( $value, $post, $field_name ) {
                 if ( 'returnError' === $value ) {
                         return new WP_Error( 'rest_invalid_param', 'Testing an error.', array( 'status' => 400 ) );
                 }
-                update_post_meta( $post->ID, 'my_custom_int', $value );
+                update_post_meta( $post->ID, $field_name, $value );
         }