Shell Programming and Scripting

#!/bin/bash
repository=$1;
txn=$2;
#echo -e "\n\npre-hook Script called\n\n";
logmsg=$(svnlook log ${repository} -t ${txn});
logmsg_vals="${logmsg%%|*}|";
logmsg_char="${logmsg_vals:0:1}";
logmsg_len=$(echo ${logmsg} | cut -d' ' -f1);           #
logmsg_len2=$(echo ${logmsg} | awk '{print $2}');       #
#logmsg_len="${logmsg%%|*}";
logmsg_suffix="|";

#echo -------------
#echo "message=$logmsg:char=$logmsg_char:value=$logmsg_len:length=${#logmsg_len}:len2=$logmsg_len2:suffix=$logmsg_suffix" >&2;
#echo -------------
#echo -e "\n $logmsg \n ${logmsg:0:1}"
  if [[ $logmsg_char == "D" || $logmsg_char == "S" ]]; then
        if  [[ "${#logmsg_len}" -le "6" ]]; then
            if [[  "${logmsg_len2}" != "${logmsg_suffix}" ]]; then
                 echo -e "\nERROR - Comment:\n\tDefect|Story number should be seperated by a PIPE character.\n\tEx: \"<D/S>12345 | <description>\"\n\nYour Comment:\n\t\"$logmsg\"\n" >&2;
                 exit 1;
            else
                echo -e "\nSuccess\n" >&2;
                exit 0;
            fi
        else
            echo -e "\nERROR - Comment:\n\tDefect|Story number (including \"D\" or \"S\" character) should be 6 chars max, excluding PIPE (\"|\") character\n\tEx. \"<D|S>12345 | <description>\"\n\nYour Comment:\n\t\"$logmsg\"\n" >&2;
#            echo -e "Your Comment:\n\t$logmsg\n" >&2;
            exit 1;
        fi
  else
     echo  -e "\nERROR - Comment:\n\tDefect|Story number that should start with either \"D\" or \"S\" character\n\tEx. \"<D|S>12345 | <description>\"\n\nYour Comment:\n\t\"$logmsg\"\n" >&2;
#     echo -e "Your Comment:\n\t\"$logmsg\"\n" >&2;
     exit 1;
  fi