Avoid blowing up if null string are passes to Escape

andreasohlund · andreasohlund · commit f019dc465e34 · 2014-03-07T10:12:18.000+01:00
diff --git a/GitFlowVersion/BuildServers/TeamCity.cs b/GitFlowVersion/BuildServers/TeamCity.cs
@@ -32,6 +32,10 @@ public string GenerateSetVersionMessage(string versionToUseForBuildNumber)
 
         static string EscapeValue(string value)
         {
+            if (value == null)
+            {
+                return null;
+            }
             // List of escape values from http://confluence.jetbrains.com/display/TCD8/Build+Script+Interaction+with+TeamCity
 
             value = value.Replace("|", "||");

Original file line number	Diff line number	Diff line change
`@@ -32,6 +32,10 @@ public string GenerateSetVersionMessage(string versionToUseForBuildNumber)`
`32`	`32`
`33`	`33`	`static string EscapeValue(string value)`
`34`	`34`	`{`
	`35`	`+ if (value == null)`
	`36`	`+ {`
	`37`	`+ return null;`
	`38`	`+ }`
`35`	`39`	`// List of escape values from http://confluence.jetbrains.com/display/TCD8/Build+Script+Interaction+with+TeamCity`
`36`	`40`
`37`	`41`	`value = value.Replace("\|", "\|\|");`