introduce parameter: ignore parts of qualified name (IDEA-104485)

java/java-impl/src/com/intellij/refactoring/introduceVariable/IntroduceVariableBase.java

@@ -197,12 +197,22 @@ public abstract class IntroduceVariableBase extends IntroduceHandlerBase {
     PsiExpression expression = PsiTreeUtil.getParentOfType(elementAtCaret, PsiExpression.class);
     while (expression != null) {
       if (!expressions.contains(expression) && !(expression instanceof PsiParenthesizedExpression) && !(expression instanceof PsiSuperExpression) && expression.getType() != PsiType.VOID) {
-        if (expression instanceof PsiMethodReferenceExpression || 
-            !(expression instanceof PsiReferenceExpression &&
-             (expression.getParent() instanceof PsiMethodCallExpression || ((PsiReferenceExpression)expression).resolve() instanceof PsiClass)) &&
-            !(expression instanceof PsiAssignmentExpression)) {
+        if (expression instanceof PsiMethodReferenceExpression) {
           expressions.add(expression);
         }
+        else if (!(expression instanceof PsiAssignmentExpression)) {
+          if (!(expression instanceof PsiReferenceExpression)) {
+            expressions.add(expression);
+          }
+          else {
+            if (!(expression.getParent() instanceof PsiMethodCallExpression)) {
+              final PsiElement resolve = ((PsiReferenceExpression)expression).resolve();
+              if (!(resolve instanceof PsiClass) && !(resolve instanceof PsiPackage)) {
+                expressions.add(expression);
+              }
+            }
+          }
+        }
       }
       expression = PsiTreeUtil.getParentOfType(expression, PsiExpression.class);
     }

java/java-tests/testData/refactoring/introduceParameter/afterPackageReferenceShouldBeIgnored.java

@@ -0,0 +1,8 @@
+import java.io.File;
+
+class Test {
+    public void foo(final File anObject) {
+        bar(anObject);
+    }
+    void bar(java.io.File f){}
+}

java/java-tests/testData/refactoring/introduceParameter/beforePackageReferenceShouldBeIgnored.java

@@ -0,0 +1,6 @@
+class Test {
+    public void foo() {
+        bar(<caret>java.io.File.createTempFile("a", "b"));
+    }
+    void bar(java.io.File f){}
+}

java/java-tests/testSrc/com/intellij/refactoring/IntroduceParameterTest.java

@@ -316,6 +316,10 @@ public class IntroduceParameterTest extends LightRefactoringTestCase  {
     doTest(IntroduceParameterRefactoring.REPLACE_FIELDS_WITH_GETTERS_ALL, true, false, true, false);
   }
 
+  public void testPackageReferenceShouldBeIgnored() throws Exception {
+    doTest(IntroduceParameterRefactoring.REPLACE_FIELDS_WITH_GETTERS_ALL, true, false, true, false);
+  }
+
   private void doTestThroughHandler() throws Exception {
     configureByFile("/refactoring/introduceParameter/before" + getTestName(false) + ".java");
     boolean enabled = true;