Merge pull request #158 from mguinness/master

damienbod · web-flow · commit 08309cb84be7 · 2018-10-24T07:03:54.000+02:00
Fixed bug in removing trailing delimiter
diff --git a/src/WebApiContrib.Core.Formatter.Csv/CsvOutputFormatter.cs b/src/WebApiContrib.Core.Formatter.Csv/CsvOutputFormatter.cs
@@ -149,7 +149,7 @@ public async override Task WriteResponseBodyAsync(OutputFormatterWriteContext co
                     }
                 }
 
-                await streamWriter.WriteLineAsync(valueLine.TrimEnd(_options.CsvDelimiter.ToCharArray()));
+                await streamWriter.WriteLineAsync(valueLine.Remove(valueLine.Length - _options.CsvDelimiter.Length));
             }
 
             await streamWriter.FlushAsync();

Original file line number	Diff line number	Diff line change
`@@ -149,7 +149,7 @@ public async override Task WriteResponseBodyAsync(OutputFormatterWriteContext co`
`149`	`149`	`}`
`150`	`150`	`}`
`151`	`151`
`152`		`- await streamWriter.WriteLineAsync(valueLine.TrimEnd(_options.CsvDelimiter.ToCharArray()));`
	`152`	`+ await streamWriter.WriteLineAsync(valueLine.Remove(valueLine.Length - _options.CsvDelimiter.Length));`
`153`	`153`	`}`
`154`	`154`
`155`	`155`	`await streamWriter.FlushAsync();`