taking the reason as the title instead of the id (#481)

jfrog · Sep 12, 2024 · dc0cfbf · dc0cfbf
1 parent 8915a4d
commit dc0cfbf
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/src/main/java/com/jfrog/ide/idea/scan/SastScannerExecutor.java b/src/main/java/com/jfrog/ide/idea/scan/SastScannerExecutor.java
@@ -46,9 +46,9 @@ List<FileTreeNode> createSpecificFileIssueNodes(List<JFrogSecurityWarning> warni
                 results.put(warning.getFilePath(), fileNode);
             }
 
-            FileIssueNode issueNode = new SastIssueNode(warning.getRuleID(),
+            FileIssueNode issueNode = new SastIssueNode(warning.getReason(),
                     warning.getFilePath(), warning.getLineStart(), warning.getColStart(), warning.getLineEnd(), warning.getColEnd(),
-                    warning.getScannerSearchTarget(), warning.getLineSnippet(), warning.getCodeFlows(), warning.getSeverity(), warning.getRuleID());
+                    warning.getReason(), warning.getLineSnippet(), warning.getCodeFlows(), warning.getSeverity(), warning.getRuleID());
             fileNode.addIssue(issueNode);
         }
         return new ArrayList<>(results.values());

diff --git a/src/main/java/com/jfrog/ide/idea/ui/webview/WebviewObjectConverter.java b/src/main/java/com/jfrog/ide/idea/ui/webview/WebviewObjectConverter.java
@@ -52,10 +52,10 @@ public static DependencyPage convertIssueToDepPage(VulnerabilityNode vulnerabili
 
     public static IssuePage convertFileIssueToIssuePage(FileIssueNode fileIssueNodeNode) {
         return new IssuePage()
+                .header(fileIssueNodeNode.getReason())
                 .type(ConvertPageType(fileIssueNodeNode.getReporterType()))
                 .severity(fileIssueNodeNode.getSeverity().name())
                 .description(fileIssueNodeNode.getReason())
-                .header(fileIssueNodeNode.getTitle())
                 .location(convertFileLocation(fileIssueNodeNode));
     }