From 7854a0cae7fa7a5f3feac66d27bae2dc42d1067a Mon Sep 17 00:00:00 2001
From: Shivam Mathur <shivam_jpr@hotmail.com>
Date: Wed, 9 Feb 2022 03:27:03 +0530
Subject: [PATCH] Fix adding composer_bin to PATH

---
 src/scripts/tools/add_tools.ps1 | 2 +-
 src/scripts/tools/add_tools.sh  | 2 +-
 src/scripts/unix.sh             | 2 +-
 src/scripts/win32.ps1           | 2 +-
 4 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/src/scripts/tools/add_tools.ps1 b/src/scripts/tools/add_tools.ps1
index 127b9c6c..a3b7a739 100644
--- a/src/scripts/tools/add_tools.ps1
+++ b/src/scripts/tools/add_tools.ps1
@@ -23,6 +23,7 @@ Function Edit-ComposerConfig() {
     Set-Content -Path $composer_json -Value "{}"
   }
   Add-EnvPATH $src\configs\composer.env
+  Add-Path $composer_bin
   if (Test-Path env:COMPOSER_TOKEN) {
     Add-Env COMPOSER_AUTH ('{"github-oauth": {"github.com": "' + $env:COMPOSER_TOKEN + '"}}')
   }
@@ -175,7 +176,6 @@ Function Add-ComposertoolHelper() {
       Remove-Item -Path $composer_lock -Force
     }
     composer global require $prefix$release $composer_args >$null 2>&1
-    Add-Path $composer_bin
     return composer global show $prefix$tool 2>&1 | findstr '^versions'
   } else {
     $release_stream = [System.IO.MemoryStream]::New([System.Text.Encoding]::ASCII.GetBytes($release))
diff --git a/src/scripts/tools/add_tools.sh b/src/scripts/tools/add_tools.sh
index bcc2b7d9..e0900541 100644
--- a/src/scripts/tools/add_tools.sh
+++ b/src/scripts/tools/add_tools.sh
@@ -43,6 +43,7 @@ configure_composer() {
     chmod 644 "$composer_json"
   fi
   add_env_path "${src:?}"/configs/composer.env
+  add_path "$composer_bin"
   if [ -n "$COMPOSER_TOKEN" ]; then
     add_env COMPOSER_AUTH '{"github-oauth": {"github.com": "'"$COMPOSER_TOKEN"'"}}'
   fi
@@ -137,7 +138,6 @@ add_composertool_helper() {
     sudo rm -f "$composer_lock" >/dev/null 2>&1 || true
     composer global require "$prefix$release" "$composer_args" >/dev/null 2>&1
     composer global show "$prefix$tool" 2>&1 | grep -E ^versions | sudo tee /tmp/composer.log >/dev/null 2>&1
-    add_path "$composer_bin"
   else
     scoped_dir="$composer_bin/_tools/$tool-$(echo -n "$release" | shasum -a 256 | cut -d ' ' -f 1)"
     if ! [ -d "$scoped_dir" ]; then
diff --git a/src/scripts/unix.sh b/src/scripts/unix.sh
index ae89ba8d..941c0e2d 100644
--- a/src/scripts/unix.sh
+++ b/src/scripts/unix.sh
@@ -99,7 +99,7 @@ get_shell_profile() {
 # Function to add a path to the PATH variable.
 add_path() {
   path_to_add=$1
-  { [ ! -d "$path_to_add" ] || [[ ":$PATH:" == *":$path_to_add:"* ]]; } && return
+  [[ ":$PATH:" == *":$path_to_add:"* ]] && return
   if [[ -n "$GITHUB_PATH" ]]; then
     echo "$path_to_add" | tee -a "$GITHUB_PATH" >/dev/null 2>&1
   else
diff --git a/src/scripts/win32.ps1 b/src/scripts/win32.ps1
index 020a1ced..d7b4737d 100644
--- a/src/scripts/win32.ps1
+++ b/src/scripts/win32.ps1
@@ -83,7 +83,7 @@ Function Add-Path {
   param(
     [string]$PathItem
   )
-  if(-not(Test-Path $PathItem) -or "$env:PATH;".contains("$PathItem;")) {
+  if("$env:PATH;".contains("$PathItem;")) {
     return
   }
   if ($env:GITHUB_PATH) {