Merge pull request #11 from intersystems/minor-bug-fix

isc-svelury · web-flow · commit eef1b4b5c65a · 2021-10-05T10:55:35.000-04:00
Minor bug fix to check for data in args before referencing it
diff --git a/cls/SourceControl/Git/Utils.cls b/cls/SourceControl/Git/Utils.cls
@@ -1105,8 +1105,12 @@ ClassMethod RunGitCommandWithInput(command As %String, inFile As %String = "", O
 
     for i=1:1:$g(args) 
     {
-        set newArgs($i(newArgs)) = args(i)
+        if ($DATA(args(i))) {
+            set newArgs($i(newArgs)) = args(i)
+        }
+        
     }
+    
     set outLog = ##class(%Library.File).TempFilename()
     set errLog = ##class(%Library.File).TempFilename()
     set returnCode = $zf(-100,"/STDOUT="_$$$QUOTE(outLog)_" /STDERR="_$$$QUOTE(errLog)_$Case(inFile, "":"", :" /STDIN="_inFile),"git",newArgs...)

Original file line number	Diff line number	Diff line change
`@@ -1105,8 +1105,12 @@ ClassMethod RunGitCommandWithInput(command As %String, inFile As %String = "", O`
`1105`	`1105`
`1106`	`1106`	`for i=1:1:$g(args)`
`1107`	`1107`	`{`
`1108`		`- set newArgs($i(newArgs)) = args(i)`
	`1108`	`+ if ($DATA(args(i))) {`
	`1109`	`+ set newArgs($i(newArgs)) = args(i)`
	`1110`	`+ }`
	`1111`	`+`
`1109`	`1112`	`}`
	`1113`	`+`
`1110`	`1114`	`set outLog = ##class(%Library.File).TempFilename()`
`1111`	`1115`	`set errLog = ##class(%Library.File).TempFilename()`
`1112`	`1116`	`set returnCode = $zf(-100,"/STDOUT="_$$$QUOTE(outLog)_" /STDERR="_$$$QUOTE(errLog)_$Case(inFile, "":"", :" /STDIN="_inFile),"git",newArgs...)`