Merge pull request #122 from CAST-projects/local_master

REPORTGEN-282 : fix issue with TECHNOLOGIES=ALL
CAST-projects · Feb 12, 2018 · b66f7fd · b66f7fd
2 parents 377b397 + 37fc9cb
commit b66f7fd
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/CastReporting.Reporting/Block/Table/ViolationsListByBC.cs b/CastReporting.Reporting/Block/Table/ViolationsListByBC.cs
@@ -24,7 +24,7 @@ public override TableDefinition Content(ReportData reportData, Dictionary<string
             string filter = options.GetOption("FILTER", "ALL");
             bool critical = options.GetOption("VIOLATIONS", "CRITICAL").Equals("CRITICAL");
             string module = options.GetOption("MODULE");
-            string[] technos = options.GetOption("TECHNOLOGIES") != null ? options.GetOption("TECHNOLOGIES").Trim().Split('|') : new[] { "$all" };
+            string[] technos = (options.GetOption("TECHNOLOGIES") != null && !options.GetOption("TECHNOLOGIES").Equals("ALL")) ? options.GetOption("TECHNOLOGIES").Trim().Split('|') : new[] { "$all" };
 
             rowData.Add(Labels.ViolationStatus);
             if (hasPri) rowData.Add(Labels.PRI);