Fixed #92: Improved total lines metric on summary page

Author: danielpalme

Readme.txt

@@ -61,6 +61,10 @@ For further details take a look at LICENSE.txt.
 
 CHANGELOG
 
+2.5.9.0
+
+    * Fix: Issue #92: Improved total lines metric on summary page
+
 2.5.8.0
 
     * New: Issue #90: Added report type "HtmlInline"

ReportGenerator.Reporting/Properties/AssemblyInfo.cs

@@ -32,7 +32,7 @@
 // You can specify all the values or you can default the Build and Revision Numbers 
 // by using the '*' as shown below:
 // [assembly: AssemblyVersion("1.0.*")]
-[assembly: AssemblyVersion("2.5.8.0")]
-[assembly: AssemblyFileVersion("2.5.8.0")]
+[assembly: AssemblyVersion("2.5.9.0")]
+[assembly: AssemblyFileVersion("2.5.9.0")]
 
 [assembly: CLSCompliant(true)]

ReportGenerator/Parser/Analysis/SummaryResult.cs

@@ -72,7 +72,31 @@ internal SummaryResult(IEnumerable<Assembly> assemblies, string usedParser, bool
         /// Gets the number of total lines.
         /// </summary>
         /// <value>The total lines.</value>
-        public int? TotalLines => this.Assemblies.Sum(a => a.TotalLines);
+        public int? TotalLines
+        {
+            get
+            {
+                var processedFiles = new HashSet<string>();
+                int? result = null;
+
+                foreach (var assembly in this.Assemblies)
+                {
+                    foreach (var clazz in assembly.Classes)
+                    {
+                        foreach (var file in clazz.Files)
+                        {
+                            if (!processedFiles.Contains(file.Path) && file.TotalLines.HasValue)
+                            {
+                                processedFiles.Add(file.Path);
+                                result = result.HasValue ? result + file.TotalLines : file.TotalLines;
+                            }
+                        }
+                    }
+                }
+
+                return result;
+            }
+        }
 
         /// <summary>
         /// Gets the coverage quota of the class.

ReportGenerator/Properties/AssemblyInfo.cs

@@ -33,8 +33,8 @@
 // Sie können alle Werte angeben oder die standardmäßigen Build- und Revisionsnummern 
 // übernehmen, indem Sie "*" eingeben:
 // [assembly: AssemblyVersion("1.0.*")]
-[assembly: AssemblyVersion("2.5.8.0")]
-[assembly: AssemblyFileVersion("2.5.8.0")]
+[assembly: AssemblyVersion("2.5.9.0")]
+[assembly: AssemblyFileVersion("2.5.9.0")]
 [assembly: NeutralResourcesLanguageAttribute("en-US")]
 
 [assembly: CLSCompliant(true)]

ReportGeneratorTest/Properties/AssemblyInfo.cs

@@ -31,6 +31,6 @@
 //
 // You can specify all the values or you can default the Revision and Build Numbers 
 // by using the '*' as shown below:
-[assembly: AssemblyVersion("2.5.8.0")]
-[assembly: AssemblyFileVersion("2.5.8.0")]
+[assembly: AssemblyVersion("2.5.9.0")]
+[assembly: AssemblyFileVersion("2.5.9.0")]
 [assembly: NeutralResourcesLanguageAttribute("en-US")]