Blocks: convert to module

projects/plugins/jetpack/_inc/client/writing/composing.jsx

@@ -1,4 +1,4 @@
-import { Chip, ToggleControl, getRedirectUrl } from '@automattic/jetpack-components';
+import { Chip, getRedirectUrl } from '@automattic/jetpack-components';
 import { __, _x } from '@wordpress/i18n';
 import CompactCard from 'components/card/compact';
 import { FormFieldset } from 'components/forms';
@@ -173,29 +173,27 @@ export class Composing extends React.Component {
 						} }
 					>
 						<FormFieldset>
-							<ToggleControl
-								checked={ ! this.props.getOptionValue( 'jetpack_blocks_disabled' ) }
-								toggling={ this.props.isSavingAnyOption( [ 'jetpack_blocks_disabled' ] ) }
-								onChange={ this.toggleBlocks }
-								label={
-									<>
-										<span className="jp-form-toggle-explanation">
-											{ __(
-												'Jetpack Blocks give you the power to deliver quality content that hooks website visitors without needing to hire a developer or learn a single line of code.',
-												'jetpack'
-											) }
-										</span>
-										{ ! this.props.getOptionValue( 'jetpack_blocks_disabled' ) && (
-											<span className="jp-form-setting-explanation">
-												{ __(
-													'Caution: if there are Jetpack blocks used in existing posts or pages, disabling this setting will cause those blocks to stop working.',
-													'jetpack'
-												) }
-											</span>
+							<ModuleToggle
+								slug="blocks"
+								activated={ !! this.props.getOptionValue( 'blocks' ) }
+								toggling={ this.props.isSavingAnyOption( [ 'blocks' ] ) }
+								toggleModule={ this.props.toggleModuleNow }
+							>
+								<span className="jp-form-toggle-explanation">
+									{ __(
+										'Jetpack Blocks give you the power to deliver quality content that hooks website visitors without needing to hire a developer or learn a single line of code.',
+										'jetpack'
+									) }
+								</span>
+								{ this.props.getOptionValue( 'blocks' ) && (
+									<span className="jp-form-setting-explanation">
+										{ __(
+											'Caution: if there are Jetpack blocks used in existing posts or pages, disabling this setting will cause those blocks to stop working.',
+											'jetpack'
 										) }
-									</>
-								}
-							/>
+									</span>
+								) }
+							</ModuleToggle>
 						</FormFieldset>
 					</SettingsGroup>
 					<CompactCard

projects/plugins/jetpack/changelog/add-blocks-module

@@ -0,0 +1,4 @@
+Significance: patch
+Type: enhancement
+
+Blocks: transition from an option to a module to improve caching

projects/plugins/jetpack/class.jetpack-gutenberg.php

@@ -12,6 +12,7 @@
 use Automattic\Jetpack\Connection\Manager as Connection_Manager;
 use Automattic\Jetpack\Constants;
 use Automattic\Jetpack\Current_Plan as Jetpack_Plan;
+use Automattic\Jetpack\Modules;
 use Automattic\Jetpack\My_Jetpack\Initializer as My_Jetpack_Initializer;
 use Automattic\Jetpack\Publicize\Jetpack_Social_Settings\Dismissed_Notices;
 use Automattic\Jetpack\Status;
@@ -432,8 +433,10 @@ public static function should_load() {
 			return false;
 		}
 
-		if ( get_option( 'jetpack_blocks_disabled', false ) ) {
-			return false;
+		$return = true;
+
+		if ( ! ( new Modules() )->is_active( 'blocks' ) ) {
+			$return = false;
 		}
 
 		/**
@@ -443,7 +446,7 @@ public static function should_load() {
 		 *
 		 * @param bool true Whether to load Gutenberg blocks
 		 */
-		return (bool) apply_filters( 'jetpack_gutenberg', true );
+		return (bool) apply_filters( 'jetpack_gutenberg', $return );
 	}
 
 	/**

projects/plugins/jetpack/class.jetpack.php

@@ -2488,6 +2488,19 @@ public static function activate_default_modules(
 		 */
 		do_action( 'jetpack_before_activate_default_modules', $min_version, $max_version, $other_modules, $requires_connection, $requires_user_connection );
 
+		// Special case to convert block setting to a block module.
+		$block_key = array_search( 'blocks', $modules, true );
+		if ( $block_key ) { // Only care if 'blocks' made it through the previous filters.
+			$block_option = get_option( 'jetpack_blocks_disabled', null );
+			if ( $block_option ) {
+				unset( $modules[ $block_key ] );
+			}
+
+			if ( $block_option !== null ) {
+				delete_option( 'jetpack_blocks_disabled' );
+			}
+		}
+
 		// Check each module for fatal errors, a la wp-admin/plugins.php::activate before activating.
 		if ( $send_state_messages ) {
 			self::restate();

projects/plugins/jetpack/modules/blocks.php

@@ -0,0 +1,13 @@
+<?php
+/**
+ * Module Name: Blocks
+ * Module Description: Add additional blocks to your site and post editors.
+ * Sort Order: 5
+ * First Introduced: 13.9
+ * Requires Connection: Yes
+ * Auto Activate: Yes
+ * Module Tags: blocks
+ * Feature: Writing
+ *
+ * @package automattic/jetpack
+ */

projects/plugins/jetpack/tests/php/general/test-class.jetpack-gutenberg.php

@@ -26,6 +26,7 @@ public function set_up() {
 		Jetpack_Options::update_option( 'master_user', $this->master_user_id );
 		Jetpack_Options::update_option( 'id', 1234 );
 		Jetpack_Options::update_option( 'blog_token', 'asd.asd.1' );
+		Jetpack::activate_default_modules();
 
 		add_filter( 'jetpack_set_available_extensions', array( __CLASS__, 'get_extensions_whitelist' ) );
 		delete_option( 'jetpack_excluded_extensions' );