[fix] fixperms should have a -h option and colors

src/fixperms.sh

@@ -22,36 +22,47 @@
 # Purpose of file: Fix permission, ACL and ownership of AlternC's files
 # ----------------------------------------------------------------------
 #
+red () { echo -e "\e[31m$@ \e[0m" ; }
+usage () {
+    [[ -n "$@" ]] && red "$@\n"
+    cat<<End-of-message
+Four optionals argument to chose from
+ -l string : a specific login to fix
+ -u integer : a specific uid to fix
+ -f string : a specific file to fix according to a given uid 
+ -d string : a specific subdirectory to fix according to a given uid 
 
-# four optionals argument to chose from
+ The u and l switch are used to fix a given user whole directory including his base directory ($ALTERNC_HTML/<letter>/<login>/
-# -l string : a specific login to fix
+ The f and d switch are used to fix a given file or directory under the user's base directory. They use the base directory to get the permissions they should use.
-# -u integer : a specific uid to fix
+ Be sure to have correct base directory permissions before attemplting to fix use those two switch 
-# -f string : a specific file to fix according to a given uid 
+End-of-message
-# -d string : a specific subdirectory to fix according to a given uid 
+    exit 1
-
+}
-# The u and l switch are used to fix a given user whole directory including his base directory ($ALTERNC_HTML/<letter>/<login>/
-# The f and d switch are used to fix a given file or directory under the user's base directory. They use the base directory to get the permissions they should use.
-# Be sure to have correct base directory permissions before attemplting to fix use those two switch 
-
 query="SELECT uid,login FROM membres ORDER BY login"
 sub_dir=""
 file=""
 LOCK_FIXPERMS="/etc/alternc/disable_all_fixperms"
+LOCK_FIXPERMS="/tmp/test"
 
 if [ -f "$LOCK_FIXPERMS" ] ; then
-  (
+
-  echo "    -------------    "
+  usage "
-  echo '/!\    WARNING    /!\ ' 
+------------------------------------    
-  echo "The fixperms script is disabled"
+/!\    WARNING    /!\  
-  echo "To enable it, delete $LOCK_FIXPERMS "
+The fixperms script is disabled
-  echo "    -------------    "
+To enable it, delete $LOCK_FIXPERMS 
-  ) 1>&2
+------------------------------------
-  exit 0
+"
+
 fi
 
-while getopts "l:u:f:d:" optname
+
+while getopts "hl:u:f:d:" optname
 do
   case "$optname" in
+  "h") usage
+      ;;
+
   "l")
     if [[ "$OPTARG" =~ ^[a-zA-Z0-9_]+$ ]] ; then
       query="SELECT uid,login FROM membres WHERE login LIKE '$OPTARG' ORDER BY login"
@@ -79,17 +90,14 @@ do
     echo $sub_dir
   ;;
   "?")
-    echo "Unknown option $OPTARG - stop processing"
+    usage "Unknown option $OPTARG - stop processing"
-    exit
   ;;
   ":")
-    echo "No argument value for option $OPTARG - stop processing"
+    usage "No argument value for option $OPTARG - stop processing"
-    exit
   ;;
   *)
     # Should not occur
-    echo "Unknown error while processing options"
+    usage "Unknown error while processing options"
-    exit
   ;;
   esac
 done