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

Timestamp:

01/03/2020 02:26:36 AM (5 years ago)

Author:

SergeyBiryukov

Message:

REST API: Synchronize permission checks in ::get_items_permissions_check() methods for post types, post statuses, and users:

• Only query post types with 'show_in_rest' => true instead of looping over all post types and checking the show_in_rest property separately.
• Return from the foreach() loop as soon as the permission check succeeded.

Props pbiron, TimothyBlynJacobs, SergeyBiryukov.
Fixes #49118.

File:

• trunk/src/wp-includes/rest-api/endpoints/class-wp-rest-post-types-controller.php (modified) (2 diffs)

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

@@ -82 +82 @@
     public function get_items_permissions_check( $request ) {
         if ( 'edit' === $request['context'] ) {
-            foreach ( get_post_types( array(), 'object' ) as $post_type ) {
-                if ( ! empty( $post_type->show_in_rest ) && current_user_can( $post_type->cap->edit_posts ) ) {
+            $types = get_post_types( array( 'show_in_rest' => true ), 'objects' );
+
+            foreach ( $types as $type ) {
+                if ( current_user_can( $type->cap->edit_posts ) ) {
                     return true;
                 }
@@ -103 +105 @@
      */
     public function get_items( $request ) {
-        $data = array();
-
-        foreach ( get_post_types( array(), 'object' ) as $obj ) {
-            if ( empty( $obj->show_in_rest ) || ( 'edit' === $request['context'] && ! current_user_can( $obj->cap->edit_posts ) ) ) {
+        $data  = array();
+        $types = get_post_types( array( 'show_in_rest' => true ), 'objects' );
+
+        foreach ( $types as $type ) {
+            if ( 'edit' === $request['context'] && ! current_user_can( $type->cap->edit_posts ) ) {
                 continue;
             }
 
-            $post_type          = $this->prepare_item_for_response( $obj, $request );
-            $data[ $obj->name ] = $this->prepare_response_for_collection( $post_type );
+            $post_type           = $this->prepare_item_for_response( $type, $request );
+            $data[ $type->name ] = $this->prepare_response_for_collection( $post_type );
         }