Merge branch 'fix/coberturaPath' of https://github.com/jeremybeier/ReportGenerator into develop

Author: danielpalme

src/Readme.txt

@@ -66,6 +66,7 @@ CHANGELOG
 4.8.8.0
 
     * Fix: Updated Angular Components (dropped support for IE)
+    * Fix: #418: Cobertura: Clean up directory separators before path combine
 
 4.8.7.0
 

src/ReportGenerator.Core/Parser/Preprocessing/CoberturaReportPreprocessor.cs

@@ -71,9 +71,15 @@ internal void Execute(XContainer report)
                         continue;
                     }
 
-                    string path = Path.Combine(sources[0], fileNameAttribute.Value)
+                    string sourcePath = sources[0]
                         .Replace('\\', Path.DirectorySeparatorChar)
                         .Replace('/', Path.DirectorySeparatorChar);
+
+                    string fileName = fileNameAttribute.Value
+                        .Replace('\\', Path.DirectorySeparatorChar)
+                        .Replace('/', Path.DirectorySeparatorChar);
+
+                    string path = Path.Combine(sourcePath, fileName);
                     fileNameAttribute.Value = path;
                 }
             }
@@ -90,10 +96,16 @@ internal void Execute(XContainer report)
 
                     foreach (var source in sources)
                     {
-                        string path = Path.Combine(source, fileNameAttribute.Value)
+                        string sourcePath = source
                             .Replace('\\', Path.DirectorySeparatorChar)
                             .Replace('/', Path.DirectorySeparatorChar);
 
+                        string fileName = fileNameAttribute.Value
+                            .Replace('\\', Path.DirectorySeparatorChar)
+                            .Replace('/', Path.DirectorySeparatorChar);
+
+                        string path = Path.Combine(sourcePath, fileName);
+
                         if (File.Exists(path))
                         {
                             fileNameAttribute.Value = path;