cr fixes

eyalk007 · Sep 22, 2024 · 153fa48 · 153fa48
1 parent 79cd857
commit 153fa48
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 6 deletions.
diff --git a/src/main/java/com/jfrog/ide/idea/navigation/NavigationService.java b/src/main/java/com/jfrog/ide/idea/navigation/NavigationService.java
@@ -49,7 +49,6 @@ public void addNavigation(DependencyNode treeNode, PsiElement navigationTargetEl
         }
         NavigationTarget navigationTarget = new NavigationTarget(navigationTargetElement, document.getLineNumber(navigationTargetElement.getTextOffset()), componentName);
         Set<NavigationTarget> navigationTargets = navigationMap.get(treeNode);
-
         if (navigationTargets == null) {
             navigationTargets = new HashSet<>(Collections.singletonList(navigationTarget));
 

diff --git a/src/main/java/com/jfrog/ide/idea/scan/GradleScanner.java b/src/main/java/com/jfrog/ide/idea/scan/GradleScanner.java
@@ -34,7 +34,7 @@
 import java.nio.file.Files;
 import java.nio.file.Path;
 import java.nio.file.Paths;
-import java.util.*;
+import java.util.Map;
 import java.util.concurrent.ExecutorService;
 
 import static com.jfrog.ide.common.log.Utils.logError;

diff --git a/src/main/java/com/jfrog/ide/idea/ui/LocalComponentsTree.java b/src/main/java/com/jfrog/ide/idea/ui/LocalComponentsTree.java
@@ -116,10 +116,11 @@ private void handleContextMenu(ComponentsTree tree, MouseEvent e) {
         }
         Object selected = selectedPath.getLastPathComponent();
 
+        // Create the popup menu if clicked on a package. if it's a vulnerability, create ignore rule option.
         if (selected instanceof DependencyNode) {
             DescriptorFileTreeNode descriptorFileTreeNode = (DescriptorFileTreeNode) selectedPath.getParentPath().getLastPathComponent();
             String descriptorPath =  descriptorFileTreeNode.getSubtitle();
-            this.createNodePopupMenu((DependencyNode) selected, descriptorPath);
+            createNodePopupMenu((DependencyNode) selected, descriptorPath);
         } else if (selected instanceof VulnerabilityNode) {
             createIgnoreRuleOption((VulnerabilityNode) selected, e);
         } else if (selected instanceof ApplicableIssueNode) {
@@ -272,6 +273,4 @@ public void setNoIssuesEmptyText() {
     public void setScanErrorEmptyText() {
         SwingUtilities.invokeLater(() -> getEmptyText().setText(ERROR_WHILE_SCANNING));
     }
-}
-
-
+}