Merge pull request #213 from DannyBen/refactor/flag-case-view

Refactor flag case view

Author: DannyBen

lib/bashly/views/flag/case.erb

@@ -1,30 +1,4 @@
 <%= view_marker %>
 <%= aliases.join " | " %> )
 <%= render(:conflicts).indent 2 %>
-% if arg
-  if [[ -n ${2+x} ]]; then
-<%= render(:validations).indent 4 %>
-% if repeatable
-    if [[ -z ${args[<%= name %>]+x} ]]; then
-      args[<%= name %>]="\"$2\""
-    else
-      args[<%= name %>]="${args[<%= name %>]} \"$2\""
-    fi
-% else
-    args[<%= name %>]="$2"
-% end
-    shift
-    shift
-  else
-    printf "%s\n" "<%= strings[:flag_requires_an_argument] % { name: name, usage: usage_string } %>"
-    exit 1
-  fi
-% else
-% if repeatable
-  (( args[<%= name %>]+=1 ))
-% else
-  args[<%= name %>]=1
-% end
-  shift
-% end
-  ;;
+<%= render(arg ? :case_arg : :case_no_arg).indent 2 %>

lib/bashly/views/flag/case_arg.erb

@@ -0,0 +1,19 @@
+<%= view_marker %>
+if [[ -n ${2+x} ]]; then
+<%= render(:validations).indent 2 %>
+% if repeatable
+  if [[ -z ${args[<%= name %>]+x} ]]; then
+    args[<%= name %>]="\"$2\""
+  else
+    args[<%= name %>]="${args[<%= name %>]} \"$2\""
+  fi
+% else
+  args[<%= name %>]="$2"
+% end
+  shift
+  shift
+else
+  printf "%s\n" "<%= strings[:flag_requires_an_argument] % { name: name, usage: usage_string } %>"
+  exit 1
+fi
+;;

lib/bashly/views/flag/case_no_arg.erb

@@ -0,0 +1,8 @@
+<%= view_marker %>
+% if repeatable
+(( args[<%= name %>]+=1 ))
+% else
+args[<%= name %>]=1
+% end
+shift
+;;