Changeset 43736 for branches/5.0/src/wp-includes/rest-api/endpoints/class-wp-rest-controller.php

Timestamp:

10/17/2018 02:40:23 AM (6 years ago)

Author:

danielbachhuber

Message:

REST API: Don't add fields to object when not included in ?_fields=.

In [43087], we improved REST API performance by only rendering the fields specified in the request. Similarly, any fields registered with register_rest_field() should only be rendered when included in ?_fields=.

Props dlh.
Fixes #45099.

File:

• branches/5.0/src/wp-includes/rest-api/endpoints/class-wp-rest-controller.php (modified) (1 diff)

branches/5.0/src/wp-includes/rest-api/endpoints/class-wp-rest-controller.php

@@ -386 +386 @@
         $additional_fields = $this->get_additional_fields();
 
+        $requested_fields = $this->get_fields_for_response( $request );
+
         foreach ( $additional_fields as $field_name => $field_options ) {
 
             if ( ! $field_options['get_callback'] ) {
+                continue;
+            }
+
+            if ( ! in_array( $field_name, $requested_fields, true ) ) {
                 continue;
             }