fix Windows support for installing node packages

htmlburger · Oct 25, 2018 · b1b2595 · b1b2595
1 parent 1c13587
commit b1b2595
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/src/NodePackageManagers/Proxy.php b/src/NodePackageManagers/Proxy.php
@@ -35,13 +35,14 @@ public function uninstall( $directory, OutputInterface $output, $package, $dev =
 	 * @return NodePackageManagerInterface
 	 */
 	protected function getNodePackageManager() {
+		$is_windows = strtolower( substr( PHP_OS, 0, 3 ) ) === 'win';
 		$node_package_managers = [
 			'yarn' => Yarn::class,
 			'npm' => Npm::class,
 		];
 
 		foreach ( $node_package_managers as $manager => $class ) {
-			$command = 'which ' . $manager;
+			$command = $is_windows ? 'where ' . escapeshellarg( $manager ) : 'which ' . escapeshellarg( $manager );
 
 			try {
 				$output = App::execute( $command );