From 501353a5342eff08a8ef22f5a294cdf0ad08b8d6 Mon Sep 17 00:00:00 2001
From: "Kenneth Benzie (Benie)" <benie@infektor.net>
Date: Wed, 9 Nov 2022 16:42:46 +0000
Subject: [PATCH] Add sandbox error message for invalid commands

---
 sandbox/sandbox.plugin.zsh | 24 +++++++++---------------
 1 file changed, 9 insertions(+), 15 deletions(-)

diff --git a/sandbox/sandbox.plugin.zsh b/sandbox/sandbox.plugin.zsh
index 8bfd7ee..1c83367 100644
--- a/sandbox/sandbox.plugin.zsh
+++ b/sandbox/sandbox.plugin.zsh
@@ -44,14 +44,11 @@ usage: sandbox [-h] {create,rename,destroy,enable,disable,list} ..
         fi
       done
       unset git
-
       [[ -z "$name" ]] && \
         error "missing argument <name>\n$usage" && return 1
-
       local sandbox=$SANDBOX_ROOT/$name
       [[ -d "$sandbox" ]] && \
         error "sandbox already exists $name" && return 1
-
       if [[ -n "$repo" ]]; then
         mkdir -p $SANDBOX_ROOT &> /dev/null
         git clone $repo $sandbox
@@ -61,55 +58,49 @@ usage: sandbox [-h] {create,rename,destroy,enable,disable,list} ..
         cd $sandbox
         git init &> /dev/null
       fi
-
       echo "SANDBOX_HOME=\$(dirname -- "\$0:a")" >> $sandbox/.enter
       echo "SANDBOX_NAME=$name" >> $sandbox/.enter
       _autoenv_authorized $sandbox/.enter yes
-
       echo "unset SANDBOX_NAME" >> $sandbox/.exit
       echo "unset SANDBOX_HOME" >> $sandbox/.exit
       _autoenv_authorized $sandbox/.exit yes
-
       _autoenv_enter $sandbox
       ;;
+
     rename)
       local old_name=$1 new_name=$2
       [[ -z "$old_name" ]] && \
         error "missing argument <old-name>\n$usage" && return 1
       [[ -z "$new_name" ]] && \
         error "missing argument <new-name>\n$usage" && return 1
-
       local old=$SANDBOX_ROOT/$old_name new=$SANDBOX_ROOT/$new_name
       [[ ! -d "$old" ]] && \
         error "sandbox does not exist $old_name" && return 1
       [[ -d "$new" ]] && \
         error "sandbox already exists $new_name" && return 1
-
       [[ "$PWD" = "$old"* ]] && _autoenv_exit $PWD
-
       mv $old $new
       sed -i "s/$old_name/$new_name/g" $new/.enter
       _autoenv_authorized $new/.enter yes
       _autoenv_authorized $new/.exit yes
-
       [[ "$PWD" = "$old"* ]] && cd $new
       ;;
+
     destroy)
       local name=$1
       [[ -z "$name" ]] && \
         error "missing argument <name>\n$usage" && return 1
-
       local sandbox=$SANDBOX_ROOT/$name
       [[ ! -d $sandbox ]] && \
         error "sandbox does not exist $name" && return 1
-
       [[ "$PWD" = "$sandbox"* ]] && cd ~
-
       rm -rf $sandbox
       ;;
+
     list)
       ls -1 $SANDBOX_ROOT | less -F -K -R -X
       ;;
+
     enable)
       local name=$1
       [[ -z "$name" ]] && \
@@ -118,16 +109,19 @@ usage: sandbox [-h] {create,rename,destroy,enable,disable,list} ..
       local sandbox=$SANDBOX_ROOT/$name
       [[ ! -d $sandbox ]] && \
         error "sandbox does not exist $name" && return 1
-
       export SANDBOX_RETURN=$PWD
       cd $sandbox
       ;;
+
     disable)
       [[ -z "$SANDBOX_RETURN" ]] && \
         error "sandbox is not currently active" && return 1
-
       cd $SANDBOX_RETURN
       unset SANDBOX_RETURN
       ;;
+
+    *)
+      error "invalid sandbox command: $cmd" && return 1
+      ;;
   esac
 }