Changeset 43833

Timestamp:

10/29/2018 12:23:11 AM (6 years ago)

Author:

danielbachhuber

Message:

Block Editor: Preload wp/v2/media with OPTIONS for caps check.

Also introduces a block_editor_preload_paths filter for plugins and themes to preload additional data.

Props imath, mattheu.
Fixes #45194.

Location:

branches/5.0

Files:

• src/wp-admin/edit-form-blocks.php (modified) (1 diff)
• src/wp-includes/rest-api.php (modified) (3 diffs)
• tests/phpunit/tests/rest-api.php (modified) (1 diff)

branches/5.0/src/wp-admin/edit-form-blocks.php

@@ -50 +50 @@
     sprintf( '/wp/v2/types/%s?context=edit', $post_type ),
     sprintf( '/wp/v2/users/me?post_type=%s&context=edit', $post_type ),
-);
+    array( '/wp/v2/media', 'OPTIONS' ),
+);
+
+/**
+ * Preload common data by specifying an array of REST API paths that will be preloaded.
+ *
+ * Filters the array of paths that will be preloaded.
+ *
+ * @since 5.0.0
+ *
+ * @param array  $preload_paths Array of paths to preload.
+ * @param object $post          The post resource data.
+ */
+$preload_paths = apply_filters( 'block_editor_preload_paths', $preload_paths, $post );
 
 /*

branches/5.0/src/wp-includes/rest-api.php

@@ -1344 +1344 @@
     }
 
+    $method = 'GET';
+    if ( is_array( $path ) && 2 === count( $path ) ) {
+        $method = end( $path );
+        $path   = reset( $path );
+
+        if ( ! in_array( $method, array( 'GET', 'OPTIONS' ), true ) ) {
+            $method = 'GET';
+        }
+    }
+
     $path_parts = parse_url( $path );
     if ( false === $path_parts ) {
@@ -1349 +1359 @@
     }
 
-    $request = new WP_REST_Request( 'GET', $path_parts['path'] );
+    $request = new WP_REST_Request( $method, $path_parts['path'] );
     if ( ! empty( $path_parts['query'] ) ) {
         parse_str( $path_parts['query'], $query_params );
@@ -1368 +1378 @@
         }
 
-        $memo[ $path ] = array(
-            'body'    => $data,
-            'headers' => $response->headers,
-        );
+        if ( 'OPTIONS' === $method ) {
+            $response = rest_send_allow_header( $response, $server, $request );
+
+            $memo[ $method ][ $path ] = array(
+                'body'    => $data,
+                'headers' => $response->headers,
+            );
+        } else {
+            $memo[ $path ] = array(
+                'body'    => $data,
+                'headers' => $response->headers,
+            );
+        }
     }
 

branches/5.0/tests/phpunit/tests/rest-api.php

@@ -630 +630 @@
         $this->assertTrue( is_array( rest_preload_api_request( 0, '/' ) ) );
     }
+
+    function test_rest_preload_api_request_with_method() {
+        $rest_server = $GLOBALS['wp_rest_server'];
+        $GLOBALS['wp_rest_server'] = null;
+
+        $preload_paths = array(
+            '/wp/v2/types',
+            array( '/wp/v2/media', 'OPTIONS' ),
+        );
+
+        $preload_data = array_reduce(
+            $preload_paths,
+            'rest_preload_api_request',
+            array()
+        );
+
+        $this->assertSame( array_keys( $preload_data ), array( '/wp/v2/types', 'OPTIONS' ) );
+        $this->assertTrue( isset( $preload_data['OPTIONS']['/wp/v2/media'] ) );
+
+        $GLOBALS['wp_rest_server'] = $rest_server;
+    }
 }