Assets: do not use the new method yet

This reverts part of #21689 Given the issues that were raised post merge on WordPress.com, let's refrain from using the new method just yet.
Automattic · Nov 16, 2021 · 9df2113 · 9df2113
1 parent b858495
commit 9df2113
Show file tree

Hide file tree

Showing 17 changed files with 208 additions and 105 deletions.
diff --git a/projects/packages/connection-ui/changelog/revert-assets-change b/projects/packages/connection-ui/changelog/revert-assets-change
@@ -0,0 +1,4 @@
+Significance: patch
+Type: fixed
+
+Assets: revert use of the new method added to register scripts
diff --git a/projects/packages/connection-ui/composer.json b/projects/packages/connection-ui/composer.json
@@ -4,7 +4,6 @@
 	"type": "library",
 	"license": "GPL-2.0-or-later",
 	"require": {
-		"automattic/jetpack-assets": "^1.12",
 		"automattic/jetpack-connection": "^1.30",
 		"automattic/jetpack-constants": "^1.6",
 		"automattic/jetpack-device-detection": "^1.4",

diff --git a/projects/packages/connection-ui/src/class-admin.php b/projects/packages/connection-ui/src/class-admin.php
@@ -7,8 +7,6 @@
 
 namespace Automattic\Jetpack\ConnectionUI;
 
-use Automattic\Jetpack\Assets;
-
 /**
  * The Connection UI Admin Area
  */
@@ -62,17 +60,14 @@ public function register_submenu_page() {
 	 */
 	public function enqueue_scripts( $hook ) {
 		if ( strpos( $hook, 'tools_page_wpcom-connection-manager' ) === 0 ) {
-			Assets::register_script(
-				'jetpack_connection_ui',
-				'../build/index.js',
-				__FILE__,
-				array(
-					'in_footer' => true,
-				)
-			);
-			Assets::enqueue_script( 'jetpack_connection_ui' );
-			wp_add_inline_script( 'jetpack_connection_ui', $this->get_initial_state(), 'before' );
-			wp_set_script_translations( 'jetpack_connection_ui', 'jetpack' );
+			$build_assets = require_once __DIR__ . '/../build/index.asset.php';
+			wp_enqueue_script( 'jetpack_connection_ui_script', plugin_dir_url( __DIR__ ) . 'build/index.js', $build_assets['dependencies'], $build_assets['version'], true );
+
+			wp_set_script_translations( 'react-jetpack_connection_ui_script', 'jetpack' );
+			wp_add_inline_script( 'jetpack_connection_ui_script', $this->get_initial_state(), 'before' );
+
+			wp_enqueue_style( 'jetpack_connection_ui_style', plugin_dir_url( __DIR__ ) . 'build/index.css', array( 'wp-components' ), $build_assets['version'] );
+			wp_style_add_data( 'jetpack_connection_ui_style', 'rtl', plugin_dir_url( __DIR__ ) . 'build/index.rtl.css' );
 		}
 	}
 

diff --git a/projects/packages/lazy-images/changelog/revert-assets-change b/projects/packages/lazy-images/changelog/revert-assets-change
@@ -0,0 +1,4 @@
+Significance: patch
+Type: fixed
+
+Assets: revert use of the new method added to register scripts
diff --git a/projects/packages/lazy-images/src/lazy-images.php b/projects/packages/lazy-images/src/lazy-images.php
@@ -485,26 +485,33 @@ private static function build_attributes_string( $attributes ) {
 	 * @return void
 	 */
 	public function enqueue_assets() {
-		Assets::register_script(
+		$script_path       = Assets::get_file_url_for_environment( '../dist/intersection-observer.min.js', '../dist/intersection-observer.src.js', __FILE__ );
+		$script_asset_path = '../dist/intersection-observer.min.assets.php';
+		$script_asset      = file_exists( $script_asset_path ) ? require $script_asset_path : array(
+			'dependencies' => array(),
+			'version'      => filemtime( __DIR__ . '/../dist/intersection-observer.min.js' ),
+		);
+		wp_enqueue_script(
 			'jetpack-lazy-images-polyfill-intersectionobserver',
-			'../dist/intersection-observer.min.js',
-			__FILE__,
-			array(
-				'nonmin_path' => '../dist/intersection-observer.src.js',
-				'in_footer'   => true,
-			)
+			$script_path,
+			$script_asset['dependencies'],
+			$script_asset['version'],
+			true
 		);
-		Assets::register_script(
+
+		$script_path       = Assets::get_file_url_for_environment( '../dist/lazy-images.min.js', 'js/lazy-images.js', __FILE__ );
+		$script_asset_path = '../dist/lazy-images.min.assets.php';
+		$script_asset      = file_exists( $script_asset_path ) ? require $script_asset_path : array(
+			'dependencies' => array(),
+			'version'      => filemtime( __DIR__ . '/../dist/lazy-images.min.js' ),
+		);
+		wp_enqueue_script(
 			'jetpack-lazy-images',
-			'../dist/lazy-images.min.js',
-			__FILE__,
-			array(
-				'nonmin_path'  => 'js/lazy-images.js',
-				'dependencies' => array( 'jetpack-lazy-images-polyfill-intersectionobserver' ),
-				'in_footer'    => true,
-			)
+			$script_path,
+			array_merge( $script_asset['dependencies'], array( 'jetpack-lazy-images-polyfill-intersectionobserver' ) ),
+			$script_asset['version'],
+			true
 		);
-		Assets::enqueue_script( 'jetpack-lazy-images' );
 		wp_localize_script(
 			'jetpack-lazy-images',
 			'jetpackLazyImagesL10n',

diff --git a/projects/plugins/backup/changelog/revert-assets-change b/projects/plugins/backup/changelog/revert-assets-change
@@ -0,0 +1,4 @@
+Significance: patch
+Type: fixed
+
+Assets: revert use of the new method added to register scripts
diff --git a/projects/plugins/backup/composer.json b/projects/plugins/backup/composer.json
@@ -4,7 +4,6 @@
 	"type": "library",
 	"license": "GPL-2.0-or-later",
 	"require": {
-		"automattic/jetpack-assets": "1.12.x-dev",
 		"automattic/jetpack-admin-ui": "0.1.x-dev",
 		"automattic/jetpack-autoloader": "2.10.x-dev",
 		"automattic/jetpack-backup": "1.1.x-dev",

diff --git a/projects/plugins/backup/composer.lock b/projects/plugins/backup/composer.lock
diff --git a/projects/plugins/backup/src/php/class-jetpack-backup.php b/projects/plugins/backup/src/php/class-jetpack-backup.php
@@ -10,7 +10,6 @@
 }
 
 use Automattic\Jetpack\Admin_UI\Admin_Menu;
-use Automattic\Jetpack\Assets;
 use Automattic\Jetpack\Connection\Manager as Connection_Manager;
 use Automattic\Jetpack\Connection\Rest_Authentication as Connection_Rest_Authentication;
 
@@ -83,16 +82,36 @@ public function admin_init() {
 	 * Enqueue plugin admin scripts and styles.
 	 */
 	public function enqueue_admin_scripts() {
-		Assets::register_script(
-			'jetpack-backup',
-			'build/index.js',
-			JETPACK_BACKUP_PLUGIN_ROOT_FILE,
-			array( 'in_footer' => true )
+		$build_assets = require_once JETPACK_BACKUP_PLUGIN_DIR . '/build/index.asset.php';
+
+		// Main JS file.
+		wp_register_script(
+			'jetpack-backup-script',
+			plugins_url( 'build/index.js', JETPACK_BACKUP_PLUGIN_ROOT_FILE ),
+			$build_assets['dependencies'],
+			$build_assets['version'],
+			true
 		);
-		Assets::enqueue_script( 'jetpack-backup' );
+		wp_enqueue_script( 'jetpack-backup-script' );
 		// Initial JS state including JP Connection data.
-		wp_add_inline_script( 'jetpack-backup', $this->get_initial_state(), 'before' );
-		wp_set_script_translations( 'jetpack-backup', 'jetpack-backup' );
+		wp_add_inline_script( 'jetpack-backup-script', $this->get_initial_state(), 'before' );
+
+		// Translation assets.
+		wp_set_script_translations( 'jetpack-backup-script-translations', 'jetpack-backup' );
+
+		// Main CSS file.
+		wp_enqueue_style(
+			'jetpack-backup-style',
+			plugins_url( 'build/index.css', JETPACK_BACKUP_PLUGIN_ROOT_FILE ),
+			array( 'wp-components' ),
+			$build_assets['version']
+		);
+		// RTL CSS file.
+		wp_style_add_data(
+			'jetpack-backup-style',
+			'rtl',
+			plugins_url( 'build/index.rtl.css', JETPACK_BACKUP_PLUGIN_ROOT_FILE )
+		);
 	}
 
 	/**

diff --git a/projects/plugins/boost/changelog/revert-assets-change b/projects/plugins/boost/changelog/revert-assets-change
@@ -0,0 +1,4 @@
+Significance: patch
+Type: fixed
+
+Assets: revert use of the new method added to register scripts
diff --git a/projects/plugins/jetpack/_inc/lib/admin-pages/class-jetpack-search-dashboard-page.php b/projects/plugins/jetpack/_inc/lib/admin-pages/class-jetpack-search-dashboard-page.php
@@ -5,7 +5,6 @@
  * @package automattic/jetpack
  */
 
-use Automattic\Jetpack\Assets;
 use Automattic\Jetpack\Status;
 
 /**
@@ -57,6 +56,7 @@ public function get_page_hook() {
 	 * Enqueue and localize page specific scripts
 	 */
 	public function page_admin_scripts() {
+		$this->load_admin_styles();
 		$this->load_admin_scripts();
 	}
 
@@ -101,27 +101,39 @@ private function get_link_offset() {
 	 * Enqueue admin styles.
 	 */
 	public function load_admin_styles() {
-		$this->load_admin_scripts();
+		\Jetpack_Admin_Page::load_wrapper_styles();
+
+		wp_enqueue_style(
+			'jp-search-dashboard',
+			plugins_url( '_inc/build/search-dashboard.css', JETPACK__PLUGIN_FILE ),
+			array(),
+			JETPACK__VERSION
+		);
 	}
 
 	/**
 	 * Enqueue admin scripts.
 	 */
 	public function load_admin_scripts() {
-		\Jetpack_Admin_Page::load_wrapper_styles();
+		$script_deps_path    = JETPACK__PLUGIN_DIR . '_inc/build/search-dashboard.asset.php';
+		$script_dependencies = array( 'react', 'react-dom', 'wp-polyfill' );
+		if ( file_exists( $script_deps_path ) ) {
+			$asset_manifest      = include $script_deps_path;
+			$script_dependencies = $asset_manifest['dependencies'];
+		}
 
 		if ( ! ( new Status() )->is_offline_mode() && Jetpack::is_connection_ready() ) {
 			// Required for Analytics.
 			Automattic\Jetpack\Tracking::register_tracks_functions_scripts( true );
 		}
 
-		Assets::register_script(
+		wp_enqueue_script(
 			'jp-search-dashboard',
-			'_inc/build/search-dashboard.js',
-			JETPACK__PLUGIN_FILE,
-			array( 'in_footer' => true )
+			plugins_url( '_inc/build/search-dashboard.js', JETPACK__PLUGIN_FILE ),
+			$script_dependencies,
+			JETPACK__VERSION,
+			true
 		);
-		Assets::enqueue_script( 'jp-search-dashboard' );
 
 		// Add objects to be passed to the initial state of the app.
 		// Use wp_add_inline_script instead of wp_localize_script, see https://core.trac.wordpress.org/ticket/25280.

diff --git a/projects/plugins/jetpack/changelog/revert-assets-change b/projects/plugins/jetpack/changelog/revert-assets-change
@@ -0,0 +1,5 @@
+Significance: patch
+Type: other
+Comment: Janitorial: do not use new method to register scripts just yet.
+
+
diff --git a/projects/plugins/jetpack/class.jetpack-gutenberg.php b/projects/plugins/jetpack/class.jetpack-gutenberg.php
@@ -6,7 +6,6 @@
  * @package automattic/jetpack
  */
 
-use Automattic\Jetpack\Assets;
 use Automattic\Jetpack\Blocks;
 use Automattic\Jetpack\Connection\Manager as Connection_Manager;
 use Automattic\Jetpack\Constants;
@@ -619,6 +618,7 @@ public static function enqueue_block_editor_assets() {
 			wp_enqueue_script( 'jp-tracks', '//stats.wp.com/w.js', array(), gmdate( 'YW' ), true );
 		}
 
+		$rtl              = is_rtl() ? '.rtl' : '';
 		$blocks_dir       = self::get_blocks_directory();
 		$blocks_variation = self::blocks_variation();
 
@@ -628,12 +628,27 @@ public static function enqueue_block_editor_assets() {
 			$blocks_env = '';
 		}
 
-		Assets::register_script(
+		$editor_script = plugins_url( "{$blocks_dir}editor{$blocks_env}.min.js", JETPACK__PLUGIN_FILE );
+		$editor_style  = plugins_url( "{$blocks_dir}editor{$blocks_env}.min{$rtl}.css", JETPACK__PLUGIN_FILE );
+
+		$editor_deps_path = JETPACK__PLUGIN_DIR . $blocks_dir . "editor{$blocks_env}.min.asset.php";
+		$editor_deps      = array( 'wp-polyfill' );
+		if ( file_exists( $editor_deps_path ) ) {
+			$asset_manifest = include $editor_deps_path;
+			$editor_deps    = $asset_manifest['dependencies'];
+		}
+
+		$version = Jetpack::is_development_version() && file_exists( JETPACK__PLUGIN_DIR . $blocks_dir . 'editor.min.js' )
+			? filemtime( JETPACK__PLUGIN_DIR . $blocks_dir . 'editor.min.js' )
+			: JETPACK__VERSION;
+
+		wp_enqueue_script(
 			'jetpack-blocks-editor',
-			"{$blocks_dir}editor{$blocks_env}.min.js",
-			JETPACK__PLUGIN_FILE
+			$editor_script,
+			$editor_deps,
+			$version,
+			false
 		);
-		Assets::enqueue_script( 'jetpack-blocks-editor' );
 
 		wp_localize_script(
 			'jetpack-blocks-editor',
@@ -689,6 +704,8 @@ public static function enqueue_block_editor_assets() {
 		);
 
 		wp_set_script_translations( 'jetpack-blocks-editor', 'jetpack' );
+
+		wp_enqueue_style( 'jetpack-blocks-editor', $editor_style, array(), $version );
 	}
 
 	/**