Skip to content

Commit

Permalink
#300: Fixed parsing issue for JaCoCo files
Browse files Browse the repository at this point in the history
  • Loading branch information
danielpalme committed Dec 12, 2019
1 parent 540778b commit 84b2d0e
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 1 deletion.
4 changes: 4 additions & 0 deletions src/Readme.txt
Original file line number Diff line number Diff line change
Expand Up @@ -61,6 +61,10 @@ For further details take a look at LICENSE.txt.

CHANGELOG

4.3.7.0

* Fix: Issue #300: Fixed parsing issue for JaCoCo files

4.3.6.0

* Fix: Issue #294: Fixed handling of partial classes for Cobertura coverage files
Expand Down
2 changes: 1 addition & 1 deletion src/ReportGenerator.Core/Parser/JaCoCoParser.cs
Original file line number Diff line number Diff line change
Expand Up @@ -135,7 +135,7 @@ private void ProcessClass(XElement[] modules, Assembly assembly, string classNam
var codeFile = ProcessFile(modules, @class, file);

var methodsOfFile = classes
.Where(c => c.Attribute("sourcefilename").Value.Equals(file))
.Where(c => c.Attribute("sourcefilename") != null && c.Attribute("sourcefilename").Value.Equals(file))
.Elements("method")
.ToArray();

Expand Down

0 comments on commit 84b2d0e

Please sign in to comment.