Merge pull request #32 from bmrvilela/master

Fix to ProjectHasScanRunning and GetScanCoverage
portilha · Aug 28, 2024 · 7449f4c · 7449f4c
2 parents cd2c204 + 67afa52
commit 7449f4c
Showing 1 changed file with 2 additions and 3 deletions.
diff --git a/Checkmarx.API/CxClient.cs b/Checkmarx.API/CxClient.cs
@@ -2330,7 +2330,7 @@ public Scan GetScanById(long scanId)
 
         public bool ProjectHasScanRunning(long projectId)
         {
-            return GetScansQueue(projectId).Any();
+            return GetScansQueue(projectId).Any(x => !x.CompletedOn.HasValue);
         }
 
         public ICollection<ScanQueue> GetScansQueue(long? projectId = null)
@@ -3184,11 +3184,10 @@ public double GetScanCoverage(long scanId)
             foreach (Match m in mc)
             {
                 GroupCollection groups = m.Groups;
-                double.TryParse(groups["pc"].Value.Replace(".", ","), out firstFinalScanAccuracy);
+                double.TryParse(groups["pc"].Value, NumberStyles.Float, CultureInfo.InvariantCulture, out firstFinalScanAccuracy);
             }
 
             return firstFinalScanAccuracy;
-
         }
 
         /// <summary>