don't suggest to replace qualifier with import if not accessible (IDEA-188061)

java/java-tests/testSrc/com/intellij/java/codeInsight/intention/AddImportActionTest.groovy

@@ -508,6 +508,19 @@ package com.rocket.test;
     assert myFixture.filterAvailableIntentions('Replace qualified name').isEmpty()
   }
 
+  void "test do not allow to add import on inaccessible class"() {
+    myFixture.addClass("package foo; class Foo {}")
+    myFixture.configureByText 'A.java', '''
+package a;
+/**
+ * {@link foo.Fo<caret>o}
+ */
+class A {}
+'''
+    myFixture.enableInspections(new UnnecessaryFullyQualifiedNameInspection())
+    assert myFixture.filterAvailableIntentions('Replace qualified name').isEmpty()
+  }
+
 
   void "test keep methods formatting on add import"() {
     settings.getCommonSettings(JavaLanguage.INSTANCE).ALIGN_GROUP_FIELD_DECLARATIONS = true

plugins/InspectionGadgets/src/com/siyeh/ig/style/UnnecessaryFullyQualifiedNameInspection.java

@@ -259,6 +259,7 @@ public class UnnecessaryFullyQualifiedNameInspection extends BaseInspection impl
         collectInnerClassNames(reference, references);
       }
       Collections.reverse(references);
+      PsiResolveHelper resolveHelper = JavaPsiFacade.getInstance(containingFile.getProject()).getResolveHelper();
       for (final PsiJavaCodeReferenceElement aReference : references) {
         final PsiElement referenceTarget = aReference.resolve();
         if (!(referenceTarget instanceof PsiClass)) {
@@ -269,6 +270,9 @@ public class UnnecessaryFullyQualifiedNameInspection extends BaseInspection impl
         if (qualifiedName == null) {
           continue;
         }
+        if (!resolveHelper.isAccessible(aClass, containingFile, null)) {
+          continue;
+        }
         if (!ImportUtils.nameCanBeImported(qualifiedName, reference)) {
           continue;
         }