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 result = new ArrayList(); @@ -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