From e684a05392f063633c8c12ac4c6f2fcd46c39c99 Mon Sep 17 00:00:00 2001
From: itssct <Thomas.Schmauder@bib.de>
Date: Wed, 15 Mar 2023 13:07:44 +0100
Subject: [PATCH] Dialog Fix, Version

---
 bibVPN.sh | 23 ++++++++++++++---------
 1 file changed, 14 insertions(+), 9 deletions(-)

diff --git a/bibVPN.sh b/bibVPN.sh
index 569d8ac..89f04d4 100755
--- a/bibVPN.sh
+++ b/bibVPN.sh
@@ -23,6 +23,7 @@
 ###################
 #### Variables ####
 ###################
+VERSION=1.2.1
 WORKDIR="/Users/$USER/.openconnect"
 mkdir -p $WORKDIR
 logfile="$WORKDIR/bibVPN.log"
@@ -57,8 +58,9 @@ IP=$(ifconfig | egrep -A1 "$NET_FILTER" |cut -d' ' -f2)
 #### Functions ####
 ###################
 function askFor(){
-    osascript=$1+' buttons {"Cancel","OK"} default button {"OK"} with title "Neuen User für das VPN anlegen"'
-    results=$( /usr/bin/osascript -e $osascript)
+    osascript="$1 buttons {\"Cancel\",\"OK\"} default button {\"OK\"} with title \"Neuen User für das VPN anlegen\""
+    echo $osascript
+    results=$( /usr/bin/osascript -e "$osascript")
 
     theButton=$( echo "$results" | /usr/bin/awk -F "button returned:|," '{print $2}' )
     theText=$( echo "$results" | /usr/bin/awk -F "text returned:" '{print $2}' )
@@ -74,10 +76,10 @@ function showSettings()
         echo "---"
         echo "Settings"
         echo "--$SETTINGSFILE"
-        echo "--$WORKDIR"
-        echo "--Farbige Icons aus"
-        echo "--Tunnelblick Icons aus"
+        echo "--Farbige Icons aus (not ready)"
         echo "--Neuen User anlegen| shell='$0' param1=newuser terminal=true refresh=true"
+        echo "---"
+        echo "--$(basename -- "$0") v$VERSION"
     fi
 }
 ###################
@@ -110,11 +112,14 @@ case "$1" in
         ;;
     newuser)
         echo "Sie wollen einen weiteren Benutzer für das VPN anlegen. Geben Sie dazu die folgenden Dinge ein."
-        #echo -n "Wie lautet der Name der neuen Verbindung? " ; read NEW_VPN_NAME
-        NEW_VPN_NAME=$(askFor 'display dialog "Wie lautet die E-Mail des Benutzers, der zum Verbinden verwendet werden soll?" default answer "vorname.nachname@bib.de"' )
+        askFor 'display dialog "Sie wollen einen weiteren Benutzer für das VPN anlegen. Geben Sie dazu die folgenden Dinge ein."'
+        NEW_VPN_NAME=$(askFor 'display dialog "Wie soll die neue Verbindung heißen?" default answer "bib | eduUP | FHDW"' )
+        NEW_VPN_USERNAME=$(askFor 'display dialog "Wie lautet die E-Mail des Benutzers, der zum Verbinden verwendet werden soll?" default answer "vorname.nachname@bib.de"' )
         NEW_VPN_HOST=$(askFor 'display dialog "Wie lautet der Servername?" default answer "vpn.bib.de"')
-        echo -n "Wie lautet der VPN-Server? "; read NEW_VPN_HOST
-        echo -n "Wie lautet die E-Mail des Benutzers? " ; read NEW_VPN_USERNAME
+        if [[ -z $NEW_VPN_NAME ]] ; then echo -n "Wie lautet der Name der neuen Verbindung? " ; read NEW_VPN_NAME ; fi
+        if [[ -z $NEW_VPN_HOST ]] ; then echo -n "Wie lautet der VPN-Server? "; read NEW_VPN_HOST ; fi
+        if [[ -z $NEW_VPN_USERNAME ]] ; then echo -n "Wie lautet die E-Mail des Benutzers? " ; read NEW_VPN_USERNAME ; fi
+
         NEW_VPN_PUBKEY=$(gnutls-cli --print-cert $NEW_VPN_HOST |grep -e 'pin-.*:'|awk '{$1=$1;print}')
         NEW_VPN_PASSWORD=$(security find-generic-password -g -a $NEW_VPN_USERNAME 2>&1 >/dev/null)
         if [[ $NEW_VPN_PASSWORD =~ password ]] ; then