From 3a822bb5fdc30e5b623f409af5c894fe9ad90d15 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Marc=20Cornell=C3=A0?= <marc.cornella@live.com>
Date: Sun, 19 Aug 2018 19:32:48 +0200
Subject: [PATCH] jenv: refactor and optimize logic

---
 plugins/jenv/jenv.plugin.zsh | 45 +++++++++++++++++-------------------
 1 file changed, 21 insertions(+), 24 deletions(-)

diff --git a/plugins/jenv/jenv.plugin.zsh b/plugins/jenv/jenv.plugin.zsh
index f131aa2f9..6c52635bb 100644
--- a/plugins/jenv/jenv.plugin.zsh
+++ b/plugins/jenv/jenv.plugin.zsh
@@ -1,33 +1,30 @@
-_homebrew-installed() {
-    type brew &> /dev/null
-}
-
-_jenv-from-homebrew-installed() {
-    brew --prefix jenv &> /dev/null
-}
-
 jenvdirs=("$HOME/.jenv" "/usr/local/jenv" "/opt/jenv")
-if _homebrew-installed && _jenv-from-homebrew-installed ; then
-    jenvdirs+=($(brew --prefix jenv) "${jenvdirs[@]}")
-fi
 
 FOUND_JENV=0
-for jenvdir in "${jenvdirs[@]}" ; do
-    if [ -d $jenvdir/bin -a $FOUND_JENV -eq 0 ] ; then
+for jenvdir in $jenvdirs; do
+    if [[ -d "${jenvdir}/bin" ]]; then
         FOUND_JENV=1
-        export PATH="${jenvdir}/bin:$PATH"
-        eval "$(jenv init - zsh)"
-
-        function jenv_prompt_info() {
-          echo "$(jenv version-name)"
-        }
-    fi
-    if [ -d $jenvdir/versions -a $FOUND_JENV -eq 0 ] ; then
-        export JENV_ROOT=$jenvdir
+        break
     fi
 done
-unset jenvdir
 
-if [ $FOUND_JENV -eq 0 ] ; then
+if [[ $FOUND_JENV -eq 0 ]]; then
+    if (( $+commands[brew] )) && jenvdir="$(brew --prefix jenv)"; then
+        FOUND_JENV=1
+    fi
+fi
+
+if [[ $FOUND_JENV -eq 1 ]]; then
+    export PATH="${jenvdir}/bin:$PATH"
+    eval "$(jenv init - zsh)"
+
+    function jenv_prompt_info() { jenv version-name 2>/dev/null }
+
+    if [[ -d "${jenvdir}/versions" ]]; then
+        export JENV_ROOT=$jenvdir
+    fi
+else
     function jenv_prompt_info() { echo "system: $(java -version 2>&1 | cut -f 2 -d ' ')" }
 fi
+
+unset jenvdir FOUND_JENV