Changeset 43087 for trunk/src/wp-includes/rest-api/endpoints/class-wp-rest-post-statuses-controller.php

Timestamp:

05/02/2018 01:24:30 AM (7 years ago)

Author:

pento

Message:

REST API: Filter responses based on the _fields parameter, before data is processed.

Historically, the REST API would generate the entire response object, including running expensive filters, then it would apply the _fields parameter, discarding the fields that weren't specificed.

This change causes _fields to be applied earlier, so that only requested fields are processed.

Props danielbachhuber.
See #43874.

File:

• trunk/src/wp-includes/rest-api/endpoints/class-wp-rest-post-statuses-controller.php (modified) (1 diff)

trunk/src/wp-includes/rest-api/endpoints/class-wp-rest-post-statuses-controller.php

@@ -200 +200 @@
     public function prepare_item_for_response( $status, $request ) {
 
-        $data = array(
-            'name'         => $status->label,
-            'private'      => (bool) $status->private,
-            'protected'    => (bool) $status->protected,
-            'public'       => (bool) $status->public,
-            'queryable'    => (bool) $status->publicly_queryable,
-            'show_in_list' => (bool) $status->show_in_admin_all_list,
-            'slug'         => $status->name,
-        );
+        $fields = $this->get_fields_for_response( $request );
+        $data   = array();
+
+        if ( in_array( 'name', $fields, true ) ) {
+            $data['name'] = $status->label;
+        }
+
+        if ( in_array( 'private', $fields, true ) ) {
+            $data['private'] = (bool) $status->private;
+        }
+
+        if ( in_array( 'protected', $fields, true ) ) {
+            $data['protected'] = (bool) $status->protected;
+        }
+
+        if ( in_array( 'public', $fields, true ) ) {
+            $data['public'] = (bool) $status->public;
+        }
+
+        if ( in_array( 'queryable', $fields, true ) ) {
+            $data['queryable'] = (bool) $status->publicly_queryable;
+        }
+
+        if ( in_array( 'show_in_list', $fields, true ) ) {
+            $data['show_in_list'] = (bool) $status->show_in_admin_all_list;
+        }
+
+        if ( in_array( 'slug', $fields, true ) ) {
+            $data['slug'] = $status->name;
+        }
 
         $context = ! empty( $request['context'] ) ? $request['context'] : 'view';