From bd2fcae8c9ad0bb508a727be8c43b04ffb44126d Mon Sep 17 00:00:00 2001
From: Eugene Kudelevsky <Eugene.Kudelevsky@jetbrains.com>
Date: Wed, 5 Feb 2014 18:01:47 +0400
Subject: [PATCH] CR-IA-226 ImportFilter: make targetFile parameter not null

---
 .../codeInsight/daemon/impl/quickfix/ImportClassFixBase.java | 2 +-
 .../src/com/intellij/codeInsight/ImportFilter.java           | 5 ++---
 2 files changed, 3 insertions(+), 4 deletions(-)

diff --git a/java/java-impl/src/com/intellij/codeInsight/daemon/impl/quickfix/ImportClassFixBase.java b/java/java-impl/src/com/intellij/codeInsight/daemon/impl/quickfix/ImportClassFixBase.java
index 957893ffe680..8047486d93ba 100644
--- a/java/java-impl/src/com/intellij/codeInsight/daemon/impl/quickfix/ImportClassFixBase.java
+++ b/java/java-impl/src/com/intellij/codeInsight/daemon/impl/quickfix/ImportClassFixBase.java
@@ -121,7 +121,7 @@ public abstract class ImportClassFixBase<T extends PsiElement, R extends PsiRefe
       String qName = aClass.getQualifiedName();
       if (qName != null) { //filter local classes
         if (qName.indexOf('.') == -1) continue; //do not show classes from default package)
-        if (qName.endsWith(name) && ImportFilter.shouldImport(file, qName)) {
+        if (qName.endsWith(name) && (file == null || ImportFilter.shouldImport(file, qName))) {
           if (isAccessible(aClass, myElement)) {
             classList.add(aClass);
           }
diff --git a/java/java-psi-api/src/com/intellij/codeInsight/ImportFilter.java b/java/java-psi-api/src/com/intellij/codeInsight/ImportFilter.java
index a862c9d1e2ef..536aa5ce77e9 100644
--- a/java/java-psi-api/src/com/intellij/codeInsight/ImportFilter.java
+++ b/java/java-psi-api/src/com/intellij/codeInsight/ImportFilter.java
@@ -3,7 +3,6 @@ package com.intellij.codeInsight;
 import com.intellij.openapi.extensions.ExtensionPointName;
 import com.intellij.psi.PsiFile;
 import org.jetbrains.annotations.NotNull;
-import org.jetbrains.annotations.Nullable;
 
 /**
  * @author Eugene.Kudelevsky
@@ -11,9 +10,9 @@ import org.jetbrains.annotations.Nullable;
 public abstract class ImportFilter {
   public static final ExtensionPointName<ImportFilter> EP_NAME = new ExtensionPointName<ImportFilter>("com.intellij.importFilter");
 
-  public abstract boolean shouldUseFullyQualifiedName(@Nullable PsiFile targetFile, @NotNull String classQualifiedName);
+  public abstract boolean shouldUseFullyQualifiedName(@NotNull PsiFile targetFile, @NotNull String classQualifiedName);
 
-  public static boolean shouldImport(@Nullable PsiFile targetFile, @NotNull String classQualifiedName) {
+  public static boolean shouldImport(@NotNull PsiFile targetFile, @NotNull String classQualifiedName) {
     for (ImportFilter filter : EP_NAME.getExtensions()) {
       if (filter.shouldUseFullyQualifiedName(targetFile, classQualifiedName)) {
         return false;