Merge branch 'RemoveBom' of https://github.com/ahsonkhan/ReportGenerator into develop

Author: danielpalme

src/Readme.txt

@@ -66,6 +66,7 @@ CHANGELOG
 4.6.6.0
 
     * New: #379: Added setting to allow saving report types to different directories
+    * New: #380: Removed UTF-8 BOM from all generated XML files
 
 4.6.5.0
 

src/ReportGenerator.Core/Reporting/Builders/CloverReportBuilder.cs

@@ -3,6 +3,8 @@
 using System.Globalization;
 using System.IO;
 using System.Linq;
+using System.Text;
+using System.Xml;
 using System.Xml.Linq;
 using Palmmedia.ReportGenerator.Core.Common;
 using Palmmedia.ReportGenerator.Core.Logging;
@@ -407,7 +409,16 @@ public void CreateSummaryReport(SummaryResult summaryResult)
 
             Logger.InfoFormat(Resources.WritingReportFile, targetPath);
 
-            result.Save(targetPath);
+            XmlWriterSettings settings = new XmlWriterSettings()
+            {
+                Encoding = new UTF8Encoding(false),
+                Indent = true
+            };
+
+            using (XmlWriter writer = XmlWriter.Create(targetPath, settings))
+            {
+                result.Save(writer);
+            }
         }
     }
 }

src/ReportGenerator.Core/Reporting/Builders/CoberturaReportBuilder.cs

@@ -3,6 +3,8 @@
 using System.Globalization;
 using System.IO;
 using System.Linq;
+using System.Text;
+using System.Xml;
 using System.Xml.Linq;
 using Palmmedia.ReportGenerator.Core.Common;
 using Palmmedia.ReportGenerator.Core.Logging;
@@ -254,7 +256,16 @@ public void CreateSummaryReport(SummaryResult summaryResult)
 
             Logger.InfoFormat(Resources.WritingReportFile, targetPath);
 
-            result.Save(targetPath);
+            XmlWriterSettings settings = new XmlWriterSettings()
+            {
+                Encoding = new UTF8Encoding(false),
+                Indent = true
+            };
+
+            using (XmlWriter writer = XmlWriter.Create(targetPath, settings))
+            {
+                result.Save(writer);
+            }
         }
 
         /// <summary>

src/ReportGenerator.Core/Reporting/Builders/SonarQubeBuilder.cs

@@ -3,6 +3,8 @@
 using System.Globalization;
 using System.IO;
 using System.Linq;
+using System.Text;
+using System.Xml;
 using System.Xml.Linq;
 using Palmmedia.ReportGenerator.Core.Common;
 using Palmmedia.ReportGenerator.Core.Logging;
@@ -112,7 +114,16 @@ public void CreateSummaryReport(SummaryResult summaryResult)
 
             Logger.InfoFormat(Resources.WritingReportFile, targetPath);
 
-            this.document.Save(targetPath);
+            XmlWriterSettings settings = new XmlWriterSettings()
+            {
+                Encoding = new UTF8Encoding(false),
+                Indent = true
+            };
+
+            using (XmlWriter writer = XmlWriter.Create(targetPath, settings))
+            {
+                this.document.Save(writer);
+            }
         }
 
         /// <summary>

src/ReportGenerator.Core/Reporting/Builders/XmlReportBuilder.cs

@@ -3,6 +3,8 @@
 using System.Globalization;
 using System.IO;
 using System.Linq;
+using System.Text;
+using System.Xml;
 using System.Xml.Linq;
 using Palmmedia.ReportGenerator.Core.Logging;
 using Palmmedia.ReportGenerator.Core.Parser.Analysis;
@@ -149,7 +151,16 @@ public override void CreateClassReport(Class @class, IEnumerable<FileAnalysis> f
 
             Logger.InfoFormat(Resources.WritingReportFile, targetPath);
 
-            result.Save(targetPath);
+            XmlWriterSettings settings = new XmlWriterSettings()
+            {
+                Encoding = new UTF8Encoding(false),
+                Indent = true
+            };
+
+            using (XmlWriter writer = XmlWriter.Create(targetPath, settings))
+            {
+                result.Save(writer);
+            }
         }
 
         /// <summary>

src/ReportGenerator.Core/Reporting/Builders/XmlSummaryReportBuilder.cs

@@ -3,6 +3,8 @@
 using System.Globalization;
 using System.IO;
 using System.Linq;
+using System.Text;
+using System.Xml;
 using System.Xml.Linq;
 using Palmmedia.ReportGenerator.Core.Common;
 using Palmmedia.ReportGenerator.Core.Logging;
@@ -159,7 +161,16 @@ public void CreateSummaryReport(SummaryResult summaryResult)
 
             Logger.InfoFormat(Resources.WritingReportFile, targetPath);
 
-            result.Save(targetPath);
+            XmlWriterSettings settings = new XmlWriterSettings()
+            {
+                Encoding = new UTF8Encoding(false),
+                Indent = true
+            };
+
+            using (XmlWriter writer = XmlWriter.Create(targetPath, settings))
+            {
+                result.Save(writer);
+            }
         }
 
         /// <summary>

src/ReportGenerator.Core/Reporting/History/HistoryReportGenerator.cs

@@ -2,6 +2,8 @@
 using System.Collections.Generic;
 using System.Globalization;
 using System.IO;
+using System.Text;
+using System.Xml;
 using System.Xml.Linq;
 using Palmmedia.ReportGenerator.Core.Common;
 using Palmmedia.ReportGenerator.Core.Logging;
@@ -87,7 +89,17 @@ internal void CreateReport(IEnumerable<Assembly> assemblies, DateTime executionT
             {
                 using (var stream = new MemoryStream())
                 {
-                    document.Save(stream);
+                    XmlWriterSettings settings = new XmlWriterSettings()
+                    {
+                        Encoding = new UTF8Encoding(false),
+                        Indent = true
+                    };
+
+                    using (XmlWriter writer = XmlWriter.Create(stream, settings))
+                    {
+                        document.Save(writer);
+                    }
+
                     stream.Position = 0;
                     this.historyStorage.SaveFile(stream, fileName);
                 }