From 70634cdfee9e48849f739c2fbba9d2c541327cf4 Mon Sep 17 00:00:00 2001
From: nik <Nikolay.Chashnikov@jetbrains.com>
Date: Thu, 28 May 2015 14:09:19 +0300
Subject: [PATCH] cleanup

---
 .../daemon/impl/quickfix/OrderEntryFix.java       | 15 ++++++---------
 1 file changed, 6 insertions(+), 9 deletions(-)

diff --git a/java/java-impl/src/com/intellij/codeInsight/daemon/impl/quickfix/OrderEntryFix.java b/java/java-impl/src/com/intellij/codeInsight/daemon/impl/quickfix/OrderEntryFix.java
index 2edfa9976289..1fca087d0d1c 100644
--- a/java/java-impl/src/com/intellij/codeInsight/daemon/impl/quickfix/OrderEntryFix.java
+++ b/java/java-impl/src/com/intellij/codeInsight/daemon/impl/quickfix/OrderEntryFix.java
@@ -147,7 +147,7 @@ public abstract class OrderEntryFix implements IntentionAction, LocalQuickFix {
         }
       };
       registrar.register(fix);
-      return Arrays.asList((LocalQuickFix)fix);
+      return Collections.singletonList((LocalQuickFix)fix);
     }
 
     if (isAnnotation(psiElement) && AnnotationUtil.isJetbrainsAnnotation(referenceName)) {
@@ -192,7 +192,7 @@ public abstract class OrderEntryFix implements IntentionAction, LocalQuickFix {
         }
       };
       registrar.register(fix);
-      return Arrays.asList((LocalQuickFix)fix);
+      return Collections.singletonList((LocalQuickFix)fix);
     }
 
     List<LocalQuickFix> result = new ArrayList<LocalQuickFix>();
@@ -225,14 +225,11 @@ public abstract class OrderEntryFix implements IntentionAction, LocalQuickFix {
 
           if (jar == null || libraryEntry.isModuleLevel() && !librariesToAdd.add(jar) || !librariesToAdd.add(library)) continue;
           OrderEntry entryForFile = moduleFileIndex.getOrderEntryForFile(virtualFile);
-          if (entryForFile != null) {
-            if (entryForFile instanceof ExportableOrderEntry &&
+          if (entryForFile != null &&
+              !(entryForFile instanceof ExportableOrderEntry &&
                 ((ExportableOrderEntry)entryForFile).getScope() == DependencyScope.TEST &&
-                !ModuleRootManager.getInstance(currentModule).getFileIndex().isInTestSourceContent(classVFile)) {
-            }
-            else {
-              continue;
-            }
+                !ModuleRootManager.getInstance(currentModule).getFileIndex().isInTestSourceContent(classVFile))) {
+            continue;
           }
           final OrderEntryFix fix = new OrderEntryFix() {
             @Override