Merge pull request #18 from stellarwp/fix/windows-compat

Ensure paths use `/` for Windows compatibility
stellarwp · Aug 8, 2024 · 91064b7 · 91064b7
2 parents be4448d + 9dcc5cc
commit 91064b7
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 3 deletions.
diff --git 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.6
+ * Version: 1.2.8
  * Author: StellarWP
  * Author URI: https://stellarwp.com
  */
diff --git a/src/Assets/Config.php b/src/Assets/Config.php
@@ -131,15 +131,21 @@ public static function set_relative_asset_path( string $path ) {
 	 * @return void
 	 */
 	public static function set_path( string $path ) {
-		$plugins_content_dir_position = strpos( $path, WP_PLUGIN_DIR );
+		$plugin_dir = WP_PLUGIN_DIR;
+
+		if ( DIRECTORY_SEPARATOR !== '/' ) {
+			$plugin_dir = str_replace( '/', DIRECTORY_SEPARATOR, $plugin_dir );
+		}
+
+		$plugins_content_dir_position = strpos( $path, $plugin_dir );
 		$themes_content_dir_position  = strpos( $path, get_theme_root() );
 
 		if (
 			$plugins_content_dir_position === false
 			&& $themes_content_dir_position === false
 		) {
 			// Default to plugins.
-			$path = WP_PLUGIN_DIR . $path;
+			$path = $plugin_dir . $path;
 		} elseif ( $plugins_content_dir_position !== false ) {
 			$path = substr( $path, $plugins_content_dir_position );
 		} elseif ( $themes_content_dir_position !== false ) {