From 68ed60a4c87600007942f98174efb82c743315a6 Mon Sep 17 00:00:00 2001
From: Dimitrios Pantazis <dpanta94@gmail.com>
Date: Fri, 14 Jun 2024 19:40:57 +0300
Subject: [PATCH 1/3] Replace module's type attr

---
 src/Assets/Assets.php | 23 ++++++++++++-----------
 1 file changed, 12 insertions(+), 11 deletions(-)

diff --git a/src/Assets/Assets.php b/src/Assets/Assets.php
index dbefe32..4b64e0c 100755
--- a/src/Assets/Assets.php
+++ b/src/Assets/Assets.php
@@ -459,16 +459,18 @@ public function filter_tag_async_defer( $tag, $handle ) {
 	/**
 	 * Filters the Script tags to attach type=module based on the rules we set in our Asset class.
 	 *
-	 * @param string $tag Tag we are filtering.
+	 * @since 1.0.0
+	 * @since TBD
+	 *
+	 * @param string $tag    Tag we are filtering.
 	 * @param string $handle Which is the ID/Handle of the tag we are about to print.
 	 *
 	 * @return string Script tag with the type=module
-	 * @since 1.0.0
-	 *
 	 */
 	public function filter_modify_to_module( $tag, $handle ) {
-		// Only filter for own own filters.
-		if ( ! $asset = $this->get( $handle ) ) {
+		$asset = $this->get( $handle );
+		// Only filter for our own filters.
+		if ( ! $asset ) {
 			return $tag;
 		}
 
@@ -482,16 +484,15 @@ public function filter_modify_to_module( $tag, $handle ) {
 			return $tag;
 		}
 
-		// These themes already have the `type='text/javascript'` added by WordPress core.
-		if ( ! current_theme_supports( 'html5', 'script' ) ) {
-			$replacement = 'type="module"';
-
-			return str_replace( "type='text/javascript'", $replacement, $tag );
+		// Remove the type attribute if it exists.
+		preg_match( "/ *type=['\"]{0,1}[^'\"]+['\"]{0,1}/i", $tag, $matches );
+		if ( ! empty( $matches ) ) {
+			$tag = str_replace( $matches[0], '', $tag );
 		}
 
 		$replacement = '<script type="module" ';
 
-		return str_replace( '<script ', $replacement, $tag );
+		return str_replace( '<script', $replacement, $tag );
 	}
 
 	/**

From b4fd2e785caddab82f655bf901e2cb8c4c39cf59 Mon Sep 17 00:00:00 2001
From: Dimitrios Pantazis <dpanta94@gmail.com>
Date: Fri, 14 Jun 2024 19:52:15 +0300
Subject: [PATCH 2/3] Add acceptance test

---
 tests/acceptance/EnqueueJSCest.php | 21 +++++++++++++++++++++
 1 file changed, 21 insertions(+)

diff --git a/tests/acceptance/EnqueueJSCest.php b/tests/acceptance/EnqueueJSCest.php
index 1aac8a3..495039e 100644
--- a/tests/acceptance/EnqueueJSCest.php
+++ b/tests/acceptance/EnqueueJSCest.php
@@ -58,6 +58,27 @@ public function it_should_enqueue_from_alternate_path_with_js_in_path( Acceptanc
 		$I->seeElement( 'script', [ 'src' => 'http://wordpress.test/wp-content/plugins/assets/tests/_data/other-asset-root/js/fake-alt.js?ver=1.0.0' ] );
 	}
 
+	public function it_should_enqueue_script_as_module( AcceptanceTester $I ) {
+		$code = file_get_contents( codecept_data_dir( 'enqueue-template.php' ) );
+		$code .= <<<PHP
+		add_action( 'wp_enqueue_scripts', function() {
+			Asset::add( 'fake-js', 'fake.js' )
+				->enqueue_on( 'wp_enqueue_scripts' )
+				->set_as_module( true )
+				->register();
+			Asset::add( 'fake3-js', 'fake3.js' )
+				->enqueue_on( 'wp_enqueue_scripts' )
+				->register();
+		}, 100 );
+		PHP;
+
+		$I->haveMuPlugin( 'enqueue.php', $code );
+
+		$I->amOnPage( '/' );
+		$I->seeElement( 'script', [ 'src' => 'http://wordpress.test/wp-content/plugins/assets/tests/_data/js/fake.js?ver=1.0.0', 'type' => 'module' ] );
+		$I->seeElement( 'script', [ 'src' => 'http://wordpress.test/wp-content/plugins/assets/tests/_data/js/fake3.min.js?ver=1.0.0' ] );
+	}
+
 	public function it_should_enqueue_min( AcceptanceTester $I ) {
 		$code = file_get_contents( codecept_data_dir( 'enqueue-template.php' ) );
 		$code .= <<<PHP

From d87bfa84d6b06b89b5192276eba2d3dbe989fcbc Mon Sep 17 00:00:00 2001
From: Dimitrios Pantazis <dpanta94@gmail.com>
Date: Fri, 14 Jun 2024 19:54:39 +0300
Subject: [PATCH 3/3] version bump

---
 assets.php            | 2 +-
 src/Assets/Assets.php | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/assets.php b/assets.php
index 7ea1071..9dfb619 100644
--- a/assets.php
+++ b/assets.php
@@ -2,7 +2,7 @@
 /**
  * Plugin Name: Assets
  * Description: Asset library with a plugin bootstrap file for automated testing.
- * Version: 1.2.5
+ * Version: 1.2.6
  * Author: StellarWP
  * Author URI: https://stellarwp.com
  */
diff --git a/src/Assets/Assets.php b/src/Assets/Assets.php
index 4b64e0c..4029798 100755
--- a/src/Assets/Assets.php
+++ b/src/Assets/Assets.php
@@ -460,7 +460,7 @@ public function filter_tag_async_defer( $tag, $handle ) {
 	 * Filters the Script tags to attach type=module based on the rules we set in our Asset class.
 	 *
 	 * @since 1.0.0
-	 * @since TBD
+	 * @since 1.2.6
 	 *
 	 * @param string $tag    Tag we are filtering.
 	 * @param string $handle Which is the ID/Handle of the tag we are about to print.