#218: Removed unnecessary trailing zeros in metrics

Author: danielpalme

src/Readme.txt

@@ -58,8 +58,9 @@ For further details take a look at LICENSE.txt.
 
 CHANGELOG
 
-4.0.14.0
+4.0.15.0
 
+    * Fix: Issue #218: Removed unnecessary trailing zeros in metrics
     * Fix: Issue #219: Fixed errors in Cobertura reports (not compliant with DTD)
 
 4.0.14.0

src/ReportGenerator.Core/Reporting/Builders/Rendering/HtmlRenderer.cs

@@ -514,7 +514,7 @@ public void CustomSummary(IEnumerable<Assembly> assemblies, IEnumerable<RiskHots
                 foreach (var metric in riskHotspot.StatusMetrics)
                 {
                     this.javaScriptContent.Append("      { ");
-                    this.javaScriptContent.AppendFormat("\"value\": {0},", metric.Metric.Value.HasValue ? metric.Metric.Value.Value.ToString(CultureInfo.InvariantCulture) : "null");
+                    this.javaScriptContent.AppendFormat("\"value\": {0},", metric.Metric.Value.HasValue ? metric.Metric.Value.Value.ToString("0.##", CultureInfo.InvariantCulture) : "null");
                     this.javaScriptContent.AppendFormat(" \"exceeded\": {0}", metric.Exceeded.ToString().ToLowerInvariant());
                     this.javaScriptContent.AppendLine(" },");
                 }
@@ -640,7 +640,7 @@ public void MetricsTable(Class @class)
                     {
                         this.reportTextWriter.Write(
                             "<td>{0}{1}</td>",
-                            metricValue.Value.HasValue ? metricValue.Value.Value.ToString(CultureInfo.InvariantCulture) : "-",
+                            metricValue.Value.HasValue ? metricValue.Value.Value.ToString("0.##", CultureInfo.InvariantCulture) : "-",
                             metricValue.Value.HasValue && metricValue.MetricType == MetricType.CoveragePercentual ? "%" : string.Empty);
                     }
 
@@ -695,7 +695,7 @@ public void MetricsTable(IEnumerable<MethodMetric> methodMetrics)
 
                 foreach (var metricValue in methodMetric.Metrics.Select(m => m.Value))
                 {
-                    this.reportTextWriter.Write("<td>{0}</td>", metricValue.HasValue ? metricValue.Value.ToString(CultureInfo.InvariantCulture) : "-");
+                    this.reportTextWriter.Write("<td>{0}</td>", metricValue.HasValue ? metricValue.Value.ToString("0.##", CultureInfo.InvariantCulture) : "-");
                 }
 
                 this.reportTextWriter.WriteLine("</tr>");
@@ -953,7 +953,7 @@ public void RiskHotspots(IEnumerable<RiskHotspot> riskHotspots)
                     this.reportTextWriter.WriteLine(
                         "<td class=\"{0} right\">{1}</td>",
                         statusMetric.Exceeded ? "lightred" : "lightgreen",
-                        statusMetric.Metric.Value.HasValue ? statusMetric.Metric.Value.Value.ToString(CultureInfo.InvariantCulture) : "-");
+                        statusMetric.Metric.Value.HasValue ? statusMetric.Metric.Value.Value.ToString("0.##", CultureInfo.InvariantCulture) : "-");
                 }
 
                 this.reportTextWriter.WriteLine("</tr>");