Framework: Drop server-side block serialization, wpautop

A parsed block has no awareness of where inner blocks exist in its innerHTML, so it cannot safely reserialize.

There are a few options:

- Since we merely skip wpautop for known blocks, we could avoid reserialization and return the block's original HTML verbatim if we had access to its outerHTML. See nylen/phpegjs#3

- Move wpautop behavior for freeform content to the editor client. This may align well with desires to transparently upgrade legacy paragraph content to paragraph blocks. This would also allow the server to avoid any preprocessing before showing a post on the front-end, assuming that the saved content has had wpautop applied already.

Acknowledging that this effectively reverts large parts of #2806

Author: aduth

lib/blocks.php

@@ -189,70 +189,3 @@ function do_blocks( $content ) {
 	return $content_after_blocks;
 }
 add_filter( 'the_content', 'do_blocks', 9 ); // BEFORE do_shortcode().
-
-/**
- * Given a string, returns content normalized with automatic paragraphs applied
- * to text not identified as a block. Since this executes the block parser, it
- * should not be used in a performance-critical flow such as content display.
- * Block content will not have automatic paragraphs applied.
- *
- * @since 1.7.0
- *
- * @param  string $content Original content.
- * @return string          Content formatted with automatic paragraphs applied
- *                         to unknown blocks.
- */
-function gutenberg_wpautop_block_content( $content ) {
-	$blocks = gutenberg_parse_blocks( $content );
-	foreach ( $blocks as $i => $block ) {
-		if ( isset( $block['blockName'] ) ) {
-			continue;
-		}
-
-		$content = $block['innerHTML'];
-
-		// wpautop will trim leading whitespace and return whitespace-only text
-		// as an empty string. Preserve to apply leading whitespace as prefix.
-		preg_match( '/^(\s+)/', $content, $prefix_match );
-		$prefix = empty( $prefix_match ) ? '' : $prefix_match[0];
-
-		$content = $prefix . wpautop( $content, false );
-
-		// To normalize as text where wpautop would not be applied, restore
-		// double newline to wpautop'd text if not at the end of content.
-		$is_last_block = ( count( $blocks ) === $i + 1 );
-		if ( ! $is_last_block ) {
-			$content = str_replace( "</p>\n", "</p>\n\n", $content );
-		}
-
-		$blocks[ $i ]['innerHTML'] = $content;
-	}
-
-	return gutenberg_serialize_blocks( $blocks );
-}
-
-/**
- * Filters saved post data to apply wpautop to freeform block content.
- *
- * @since 1.7.0
- *
- * @param  array $data An array of slashed post data.
- * @return array       An array of post data with wpautop applied to freeform
- *                     block content.
- */
-function gutenberg_wpautop_insert_post_data( $data ) {
-	if ( ! empty( $data['post_content'] ) && gutenberg_content_has_blocks( $data['post_content'] ) ) {
-		// WP_REST_Posts_Controller slashes post data before inserting/updating
-		// a post. This data gets unslashed by `wp_insert_post` right before
-		// saving to the DB. The PEG parser needs unslashed input in order to
-		// properly parse JSON attributes.
-		$content = wp_unslash( $data['post_content'] );
-		$content = gutenberg_wpautop_block_content( $content );
-		$content = wp_slash( $content );
-
-		$data['post_content'] = $content;
-	}
-
-	return $data;
-}
-add_filter( 'wp_insert_post_data', 'gutenberg_wpautop_insert_post_data' );

lib/client-assets.php

@@ -737,12 +737,6 @@ function gutenberg_editor_scripts_and_styles( $hook ) {
 		);
 	}
 
-	// Set initial content to apply autop on unknown blocks, preserving this
-	// behavior for classic content while otherwise disabling for blocks.
-	if ( ! $is_new_post && is_array( $post_to_edit['content'] ) ) {
-		$post_to_edit['content']['raw'] = gutenberg_wpautop_block_content( $post_to_edit['content']['raw'] );
-	}
-
 	// Set the post type name.
 	$post_type = get_post_type( $post );