Merge pull request #33 from stellarwp/feature/add_version_filter

Author: JPry

src/Assets/Asset.php

@@ -1123,7 +1123,16 @@ public function get_version(): string {
 			return (string) $asset_file_contents['version'];
 		}
 
-		return $this->version;
+		$hook_prefix = Config::get_hook_prefix();
+
+		/**
+		 * Filters the asset version when it doesn't come from an asset file.
+		 *
+		 * @param string $version The asset version.
+		 * @param string $slug    The asset slug.
+		 * @param Asset  $asset   The Asset object.
+		 */
+		return (string) apply_filters( "stellarwp/assets/{$hook_prefix}/version", $this->version, $this->slug, $this );
 	}
 
 	/**

tests/wpunit/AssetTest.php

@@ -29,4 +29,31 @@ public function test_add_to_group_path_changes_resolution_path_to_group(): void
 
 		$this->assertEquals( WP_PLUGIN_DIR . '/assets/tests/_data/', $asset->get_root_path() );
 	}
+
+	public function test_can_filter_asset_version(): void {
+		$prefix = 'bork';
+
+		Config::reset();
+		Config::set_hook_prefix( $prefix );
+		Config::set_version( '1.1.0' );
+		Config::set_path( constant( 'WP_PLUGIN_DIR' ) . '/assets' );
+
+		$asset = new Asset( 'test-script', 'fake.js', '1.0.0', codecept_data_dir() );
+
+		// The asset version should be the same as the one passed to the constructor.
+		$this->assertEquals( '1.0.0', $asset->get_version() );
+
+		// Filter the asset version.
+		$filter = function () {
+			return '2.0.0';
+		};
+
+		add_filter( "stellarwp/assets/{$prefix}/version", $filter );
+
+		// After filtering, the asset version should be updated.
+		$this->assertEquals( '2.0.0', $asset->get_version() );
+
+		// Cleanup the filter.
+		remove_filter( "stellarwp/assets/{$prefix}/version", $filter );
+	}
 }