From 19e8b8d60856d61cf0294986b22dce29d341ec64 Mon Sep 17 00:00:00 2001
From: kyodev <kyodev@yandex.fr>
Date: Sat, 23 Sep 2017 12:10:32 +0200
Subject: [PATCH] getIp 1.10.0

---
 scripts/getIp              | 63 +++++++++++++++++++++-----------------
 scripts/getIp_changelog.md | 10 ++++++
 2 files changed, 45 insertions(+), 28 deletions(-)

diff --git a/scripts/getIp b/scripts/getIp
index 2fd4c49..3b897eb 100755
--- a/scripts/getIp
+++ b/scripts/getIp
@@ -1,7 +1,7 @@
 #!/bin/bash
 
-version=1.9.0
-date="07/09/2017"
+version=1.10.0
+date="23/09/2017"
 projet="simpledeb"
 contact="IRC freenode ##sdeb ou https://framagit.org/kyodev/kyopages/issues/"
 script="getIp"
@@ -13,7 +13,7 @@ urlNotice="https://kyodev.frama.io/kyopages/scripts/getIp/"
 #    contributeurs: kyodev
 #####
 
-f__affichage(){		# v27/08/2017	définition couleurs
+f__affichage(){		# 27/08/2017	définition couleurs
 	YELLOW="$(printf "\033[0;33m")"		# ?
 	 GREEN="$(printf "\033[0;32m")"		# ok
 	  BLUE="$(printf "\033[0;34m")"		# info
@@ -22,25 +22,27 @@ f__affichage(){		# v27/08/2017	définition couleurs
 }
 
 	# affichage $1 en rouge, $1++ optionnels en bleu, sortie script sur erreur, log $1 si $opType=upgrade
-f__error() {		# v25/08/2017
+f__error() {		# 22/09/2017
 	echo -e "\n$RED  $script $version, erreur critique: $1 $COLOR"
-	for (( i=2 ; i<=$# ; i++ )); do echo -e "  $BLUE${!i}"; done 
+	for (( i=2 ; i<=$# ; i++ )); do echo -e "  $BLUE${!i}"; done
+	unset i
 	echo "$COLOR"
 	if [ "$opType" == "upgrade" ]; then f__log "$script $version: $1"; fi
 	exit 1
 }
 
 	# affichage des paramètres en bleu, si $1=raw pas de ligne vide à la fin, si $1=log uniquement $2 logué
-f__info() {		# v25/08/2017
+f__info() {		# 22/09/2017
 	local depart=1
 	if [ "$1" == "raw" ] || [ "$1" == "log" ]; then depart=2; fi
 	[ "$1" == "log" ] && f__log "$(sed -E 's/\\t//;s/\\n// ' <<< $2 | xargs )"
 	for (( i=$depart ; i<=$# ; i++ )); do echo -e "  $BLUE${!i}"; done
+	unset i
 	[ "$1" == raw ] && printf "$COLOR" || echo "$COLOR"
 }
 
 	# log spécifique, fichier log limité à 10000octets, $1 message à loguer
-f__log(){		# v08/O9/2017
+f__log(){		# 08/O9/2017
 	if [ -w "$fileLogs" ];  then 
 		if [ "$(stat -c %s  $fileLogs)" -ge "10000" ]; then
 			echo "$(date +%Y%m%d\ %H%M%S) $1"  &>/dev/null > "$fileLogs"
@@ -58,7 +60,7 @@ f__log(){		# v08/O9/2017
 	# si $2=debOnly et si paquets manquants: return 1 et $debOnlyPackages ( $1=liste paquets )
 	# si $2=debOnly et si paquets présent: return 0 et $debOnlyPresents ( $1=liste paquets )
 	# attention priorité $debOnlyPackages sur $debOnlyPresents
-f__requis(){		# v30/08/2017-3
+f__requis(){		# 22/09/2017
 	local dependsMissing packagesMissing command package
 	unset debOnlyPackages debOnlyPresents
 	for i in $1; do
@@ -74,6 +76,7 @@ f__requis(){		# v30/08/2017-3
 			packagesMissing+="$package "
 		fi
 	done
+	unset i
 	[ "$debOnlyPackages" ] && debOnlyPackages="$(xargs <<< $debOnlyPackages)"		# trim début & fin
 	[ "$debOnlyPresents" ] && debOnlyPresents="$(xargs <<< $debOnlyPresents)"		# trim début & fin
 	[ "$debOnlyPackages" ] && return 1 
@@ -88,7 +91,7 @@ f__requis(){		# v30/08/2017-3
 	# user ayant initié la session graphique, assigne $user_
 	# return 1 sur échec identification user, return 2 sur absence home/
 	# gestion variable environnement user avec: USER_INSTALL=user script
-f__user(){		# v30/08/2017-4
+f__user(){		# 30/08/2017-4
 	if [ "$USER_INSTALL" ]; then 	# user_ via variable environnement, moyen d'injecter root 
 		user_="$USER_INSTALL"; 
 		return 0
@@ -122,7 +125,7 @@ f__user(){		# v30/08/2017-4
 	# test wget, $1 url à tester, sortie script, sur erreur ou retour à la normale
 	# si $2=print affiche url testée & entêtes http & location (si présente) et sortie normale fonction
 	# si $2=loc affiche seulement location
-f__wget_test(){		# v07/09/2017
+f__wget_test(){		# 07/09/2017
 	local file_test_wget retourWget retourHttp location
 	file_test_wget="/tmp/testWget-$RANDOM"
 	wget -Sq --tries=1 --timeout=10 --user-agent="$user_agent" --spider --save-headers "$1" &>"$file_test_wget" || retourWget="$?"
@@ -164,7 +167,7 @@ f__wget_test(){		# v07/09/2017
 	exit 0
 }
 
-f_help(){		# v07/09/2017
+f_help(){		# 07/09/2017
 	f__info "\n  $GREEN$script $version$BLUE"
 cat << 'EOF'
   ----------------------------------------------------------------------
@@ -190,7 +193,7 @@ echo -e "$COLOR  plus d'infos: $GREEN$urlNotice$COLOR\n"
 
 	# assigne $ip_local, $gateway, $gateway_type, $gateway_one, $ip_local, $ip_local_type
 	# $ifnames, $ifnames_type, $mac_address, mac_address_type
-figet_ip() {		# v04/09/2017
+figet_ip() {		# 23/09/2017
 	[ -z "$(which ip)" ] && return 1
 	ip_local="$(sed '/lo.*inet/d; /inet6.*scope global/d' <<< "$(ip -o a)" | awk '{print " ",$4,"(",$2,")"}')"
 	ip_local_type="$(sed -E 's/(^.*wl.*)/\1 (wifi)/;s/(^.*en.*|^.*eth.*)/\1 (ethernet)/' <<< $ip_local)"
@@ -201,6 +204,7 @@ figet_ip() {		# v04/09/2017
 		[ "$ifn" != "lo" ] && ifnames+="  $ifn"$'\n'		# $'\n' -> newline ou $'\x0a' ou $'\012'
 		[ "$ifn" != "lo" ] && mac_address+="  $ifn: $(cat /sys/class/net/$ifn/address)"$'\n'
 	done
+	unset ifn
 	ifnames="$(sed '/^$/d' <<< $ifnames)"	# suppression \n final
 	ifnames_type+="$(sed -E 's/(^.*wl.*)/\1 (wifi)/;s/(^.*en.*|^.*eth.*)/\1 (ethernet)/' <<< $ifnames)"
 	mac_address="$(sed '/^$/d' <<< $mac_address)"	# suppression \n final
@@ -208,7 +212,7 @@ figet_ip() {		# v04/09/2017
 }
 
 	# $1=4|6, assigne $ip_public
-figet_ip_public(){		# v03/09/2017
+figet_ip_public(){		# 23/09/2017
 
 	list_ip4(){
 		ip_test+=" http://whatismyip.akamai.com"
@@ -261,26 +265,29 @@ figet_ip_public(){		# v03/09/2017
 	fi
 
 	if [ "$(which dig)" ] && [ -z "$ip_public" ]; then
-		for ii in $dig_test ; do
-			ip_public="$(dig +short $(sed 's;/; ;g' <<< $ii))"
+		for i in $dig_test ; do
+			ip_public="$(dig +short $(sed 's;/; ;g' <<< $i))"
 			[ "$ip_public" ] && break
 		done
+		unset i
 	fi
 
 	if [ "$(which wget)" ] && [ -z "$ip_public" ]; then 
 		cmd="wget --quiet --tries=1 --timeout=5 -O - "
-		for ii in $ip_test ; do
-			ip_public="$($cmd $ii)"
+		for i in $ip_test ; do
+			ip_public="$($cmd $i)"
 			[ "$ip_public" ] && break
 		done
+		unset i
 	fi
 
 	if [ "$(which curl)" ] && [ -z "$ip_public" ]; then
 		cmd="curl --silent --location --retry 0 --max-time 5"	#--location pour aider redirections
-		for ii in $ip_test ; do
-			ip_public="$($cmd $ii)"
+		for i in $ip_test ; do
+			ip_public="$($cmd $i)"
 			[ "$ip_public" ] && break
 		done
+		unset i
 	fi
 
 	if [ "$(which telnet)" ] && [ -z "$ip_public" ]; then
@@ -301,7 +308,7 @@ figet_ip_public(){		# v03/09/2017
 }
 
 	# inscription dans tache upgrade en anacron hebdomadaire, via cron horaire, $1=upgrade|install|remove
-fscript_cronAnacron(){		# v06/09/2017
+fscript_cronAnacron(){		# 06/09/2017
 	local dirAnacron dirSpool fileAnacron
 	[ "$(type -t fscript_cronAnacron_special)" ] && fscript_cronAnacron_special		# test, si fonction spécifique, appel
 	dirAnacron="/home/$user_/.config/anacron"
@@ -336,7 +343,7 @@ fscript_cronAnacron(){		# v06/09/2017
 }
 
 	# version script en ligne, assigne $versionScript, $script_aJour=ok|ko
-fscript_get_version(){		# v07/09/2017
+fscript_get_version(){		# 07/09/2017
 	f__info "raw" "$GREEN""version script en cours: $version"
 	versionScript="$(wget -q --tries=2 --timeout=15 -O - "$urlScript" | grep '^version=' | cut -d '=' -f 2)"
 	if [ "$versionScript" ]; then 
@@ -351,7 +358,7 @@ fscript_get_version(){		# v07/09/2017
 }
 
 	# installation du script dans le système
-fscript_install(){		# v06/09/2017
+fscript_install(){		# 06/09/2017
 	if grep -q 'bin' <<< "$(dirname $0)" ; then
 		f__info "$RED""l'installation dans le système doit se faire depuis un script local $GREEN(./$script -i )$COLOR"
 		return
@@ -378,7 +385,7 @@ fscript_install(){		# v06/09/2017
 }
 
 	# suppression du script dans le système
-fscript_remove(){		# v06/09/2017
+fscript_remove(){		# 06/09/2017
 	if ! grep -q 'bin' <<< "$(dirname $0)" ; then
 		f__info "$RED""cette fonction doit être appelée depuis le script installé dans le système $GREEN($script -r)$COLOR"
 		return 1
@@ -400,7 +407,7 @@ fscript_remove(){		# v06/09/2017
 }
 
 	# mise à jour script si dispo
-fscript_update(){		# v06/09/2017
+fscript_update(){		# 06/09/2017
 	if ! grep -q 'bin' <<< "$(dirname $0)" ; then
 		f__info "$RED""cette fonction doit être appelée depuis le script installé dans le système $GREEN($script -u)$COLOR"
 		return
@@ -422,7 +429,7 @@ fscript_update(){		# v06/09/2017
 	f__info "log" "$script mis à jour en version $versionScript"
 }
 
-testuser(){
+testuser(){		# 08/2017
 	f__user && retourFUser=0 || retourFUser="$?"
 	[ "$retourFUser" -eq 1 ] && f__error "user indéterminé, pour contourner, lancer le script avec:\n  USER_INSTALL=user $0 \n"
 	[ "$retourFUser" -eq 2 ] && f__error "user détecté, mais pas de home: /home/$user_"
@@ -437,8 +444,8 @@ fileDev="/opt/bin/fileDev"
 f__requis "gawk wget ip>iproute2"		# requis pour fonctionnement programme
 
 options="$@"
-for i in "$options"; do
-	case $i  in
+for j in "$options"; do
+	case $j  in
 		-4 | --ip4 )
 			if figet_ip_public "4" ; then
 				echo "$ip_public"
@@ -480,7 +487,7 @@ for i in "$options"; do
 			;;											# affichage help
 	esac
 done
-
+unset j
 exit 0
 
 wget -O getIp https://framagit.org/kyodev/kyopages/raw/master/scripts/getIp
diff --git a/scripts/getIp_changelog.md b/scripts/getIp_changelog.md
index 2f92dc1..6a8c048 100644
--- a/scripts/getIp_changelog.md
+++ b/scripts/getIp_changelog.md
@@ -1,6 +1,16 @@
 # changelog getIp
 
 
+## todo
+
+* placé ln dans /usr/local/bin au lieu de /usr/bin ?
+
+
+## 1.10.0 23/09/2017
+
+* f__requis, f__info, f__error	unset/for
+* révision unset/for
+
 ## 1.9.0 07/09/2017
 
 * correction f_help