Fixes longstanding array composition bug on zsh. (#210)

Author: marc0der

src/main/bash/sdkman-init.sh

@@ -42,11 +42,6 @@ fi
 
 export SDKMAN_CANDIDATES_DIR="${SDKMAN_DIR}/candidates"
 
-# force zsh to behave well
-if [[ -n "$ZSH_VERSION" ]]; then
-	setopt shwordsplit
-fi
-
 # OS specific support (must be 'true' or 'false').
 cygwin=false;
 darwin=false;
@@ -66,6 +61,16 @@ case "$(uname)" in
 		freebsd=true
 esac
 
+# Determine shell
+zsh_shell=false;
+bash_shell=false;
+
+if [[ -n "$ZSH_VERSION" ]]; then
+    zsh_shell=true
+else
+    bash_shell=true
+fi
+
 # Source sdkman module scripts.
 for f in $(find "${SDKMAN_DIR}/src" -type f -name 'sdkman-*' -exec basename {} \;); do
 	source "${SDKMAN_DIR}/src/${f}"
@@ -111,12 +116,6 @@ else
 	fi
 fi
 
-# Set the candidate array
-OLD_IFS="$IFS"
-IFS=","
-SDKMAN_CANDIDATES=(${SDKMAN_CANDIDATES_CSV})
-IFS="$OLD_IFS"
-
 # determine if up to date
 SDKMAN_VERSION_FILE="${SDKMAN_DIR}/var/version"
 if [[ "$sdkman_beta_channel" != "true" && -f "$SDKMAN_VERSION_FILE" && -z "$(find "$SDKMAN_VERSION_FILE" -mmin +$((60*24)))" ]]; then
@@ -144,6 +143,16 @@ else
 	fi
 fi
 
+# Set the candidate array
+if [[ "$zsh_shell" == 'true' ]]; then
+    SDKMAN_CANDIDATES=( ${(s:,:)SDKMAN_CANDIDATES_CSV} )
+else
+    OLD_IFS="$IFS"
+    IFS=","
+    SDKMAN_CANDIDATES=(${SDKMAN_CANDIDATES_CSV})
+    IFS="$OLD_IFS"
+fi
+
 # The candidates are assigned to an array for zsh compliance, a list of words is not iterable
 # Arrays are the only way, but unfortunately zsh arrays are not backward compatible with bash
 # In bash arrays are zero index based, in zsh they are 1 based(!)