Changeset 47360

Timestamp:

02/25/2020 03:03:57 PM (5 years ago)

Author:

kadamwhite

Message:

REST API: Apply all relevant block rendering filters when rendering block previews.

Several filters were introduced to the render_block method since the initial implementation of the block-renderer/ endpoints, causing the output of those endpoints to diverge from the rendered content of blocks on the frontend.

Props kadamwhite, TimothyBlynJacobs, miinasikk.
Fixes #49387.

Location:

trunk

Files:

• src/wp-includes/rest-api/endpoints/class-wp-rest-block-renderer-controller.php (modified) (2 diffs)
• tests/phpunit/tests/rest-api/rest-block-renderer-controller.php (modified) (1 diff)

trunk/src/wp-includes/rest-api/endpoints/class-wp-rest-block-renderer-controller.php

@@ -136 +136 @@
         }
         $registry = WP_Block_Type_Registry::get_instance();
-        $block    = $registry->get_registered( $request['name'] );
 
-        if ( null === $block ) {
+        if ( null === $registry->get_registered( $request['name'] ) ) {
             return new WP_Error(
                 'block_invalid',
@@ -148 +147 @@
         }
 
+        $attributes = $request->get_param( 'attributes' );
+
+        // Create an array representation simulating the output of parse_blocks.
+        $block = array(
+            'blockName'    => $request['name'],
+            'attrs'        => $attributes,
+            'innerHTML'    => '',
+            'innerContent' => array(),
+        );
+
+        // Render using render_block to ensure all relevant filters are used.
         $data = array(
-            'rendered' => $block->render( $request->get_param( 'attributes' ) ),
+            'rendered' => render_block( $block ),
         );
 

trunk/tests/phpunit/tests/rest-api/rest-block-renderer-controller.php

@@ -374 +374 @@
     }
 
-
+    /**
+     * Check filtering block output using the pre_render_block filter.
+     *
+     * @ticket 49387
+     */
+    public function test_get_item_with_pre_render_block_filter() {
+        wp_set_current_user( self::$user_id );
+
+        $pre_render_filter = function( $output, $block ) {
+            if ( $block['blockName'] === self::$block_name ) {
+                return '<p>Alternate content.</p>';
+            }
+        };
+        add_filter( 'pre_render_block', $pre_render_filter, 10, 2 );
+
+        $attributes = array(
+            'some_int'    => '123',
+            'some_string' => 'foo',
+            'some_array'  => array( 1, '2', 3 ),
+        );
+
+        $request = new WP_REST_Request( 'GET', self::$rest_api_route . self::$block_name );
+        $request->set_param( 'context', 'edit' );
+        $request->set_param( 'attributes', $attributes );
+        $response = rest_get_server()->dispatch( $request );
+        $this->assertEquals( 200, $response->get_status() );
+
+        $data = $response->get_data();
+        $this->assertEquals( '<p>Alternate content.</p>', $data['rendered'] );
+
+        remove_filter( 'pre_render_block', $pre_render_filter );
+    }
 
     /**