separate get ftproot and gvolume

gluster/grf

@@ -13,8 +13,56 @@ fatal()
     exit 1
 }
 
+get_ftproot()
+{
+    local FTPROOT
+    local sfile="$1"
+
+    # hack
+    case "$sfile" in
+    /var/ftp/pvt/*)
+        FTPROOT=/var/ftp/pvt
+        ;;
+    /var/ftp/pub/*)
+        FTPROOT=/var/ftp/pub
+        ;;
+    /var/ftp/tmp/*)
+        FTPROOT=/var/ftp/tmp
+        ;;
+    *)
+        fatal "Unknown place $sfile"
+    esac
+
+    echo "$FTPROOT"
+}
+
+get_volume()
+{
+    local GVOLUME
+    local sfile="$1"
+
+    # hack
+    case "$sfile" in
+    /var/ftp/pvt/*)
+        GVOLUME=ftp-pvt
+        ;;
+    /var/ftp/pub/*)
+        GVOLUME=ftp-pub
+        ;;
+    /var/ftp/tmp/*)
+        GVOLUME=ftp-tmp
+        ;;
+    *)
+        fatal "Unknown place $sfile"
+    esac
+
+    echo "$GVOLUME"
+}
+
+
 get_bricks()
 {
+    local GVOLUME="$1"
     gluster volume info $GVOLUME | grep Brick"[0-9]" | sed -e "s|Brick[0-9]: ||g" | sed -e "s|.(.*||g" | grep -v "arbiter"
 }
 
@@ -128,25 +176,11 @@ if [ "$1" = "--list" ] ; then
 fi
 
 
-# hack
-case "$sfile" in
-    /var/ftp/pvt/*)
-        FTPROOT=/var/ftp/pvt
-        GVOLUME=ftp-pvt
-        ;;
-    /var/ftp/pub/*)
-        FTPROOT=/var/ftp/pub
-        GVOLUME=ftp-pub
-        ;;
-    /var/ftp/tmp/*)
-        FTPROOT=/var/ftp/tmp
-        GVOLUME=ftp-tmp
-        ;;
-    *)
-        fatal "Unknown place $sfile"
-esac;
+FTPROOT=$(get_ftproot $sfile)
+GVOLUME=$(get_volume $sfile)
 
-LISTBRICKS="$(get_bricks)"
+LISTBRICKS="$(get_bricks $GVOLUME)"
+[ -n "$LISTBRICKS" ] || fatal "empty LISTBRICKS"
 
 MOUNTDIR=~/grf-mount