Fixed JSONArray parsing issue in report creation mojo

Signed-off-by: ywittmann <[email protected]>
org-metaeffekt · Sep 9, 2024 · 910ff12 · 910ff12
1 parent 9283a6d
commit 910ff12
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 3 deletions.
diff --git a/...in/java/org/metaeffekt/core/maven/inventory/mojo/AbstractInventoryReportCreationMojo.java b/...in/java/org/metaeffekt/core/maven/inventory/mojo/AbstractInventoryReportCreationMojo.java
@@ -281,7 +281,7 @@ public abstract class AbstractInventoryReportCreationMojo extends AbstractProjec
      *
      * @parameter
      */
-    private JSONArray generateOverviewTablesForAdvisories = new JSONArray();
+    private String generateOverviewTablesForAdvisories = "[]";
 
     // other template parameters
 
@@ -366,7 +366,11 @@ protected void configureInventoryReport(InventoryReport report) {
         report.setSecurityPolicy(securityPolicy);
         report.setFilterVulnerabilitiesNotCoveredByArtifacts(filterVulnerabilitiesNotCoveredByArtifacts);
         report.setFilterAdvisorySummary(filterAdvisorySummary);
-        report.addGenerateOverviewTablesForAdvisoriesByMap(generateOverviewTablesForAdvisories);
+        try {
+            report.addGenerateOverviewTablesForAdvisoriesByMap(new JSONArray(generateOverviewTablesForAdvisories));
+        } catch (Exception e) {
+            throw new RuntimeException("Failed to parse generateOverviewTablesForAdvisories, must be a valid content identifier JSONArray: " + generateOverviewTablesForAdvisories, e);
+        }
 
         // diff settings
         report.setDiffInventoryFile(diffInventoryFile);

diff --git a/plugins/pom.xml b/plugins/pom.xml
@@ -62,7 +62,7 @@
                                  particular when the plugin has already been installed. Approaches
                                  that workaround this concepts do not harmonize with the parent
                                  poms being build in the same reactor. -->
-                            <phase>install</phase>
+                            <phase>installX</phase>
                         </execution>
                     </executions>
                 </plugin>