EA-74878 - IAE: MoveClassToInnerProcessor.<init>
authorAnna Kozlova <anna.kozlova@jetbrains.com>
Wed, 28 Oct 2015 11:34:43 +0000 (12:34 +0100)
committerAnna Kozlova <anna.kozlova@jetbrains.com>
Wed, 28 Oct 2015 15:05:01 +0000 (16:05 +0100)
java/java-impl/src/com/intellij/refactoring/move/moveClassesOrPackages/MoveClassesOrPackagesDialog.java

index e0d206e733c15fcf8b06502fb2acd250103783a1..8ea8518877d8a64cb3ecfdd591bf21c759adc5cd 100644 (file)
@@ -44,7 +44,6 @@ import com.intellij.refactoring.move.MoveHandler;
 import com.intellij.refactoring.move.moveFilesOrDirectories.MoveFilesOrDirectoriesUtil;
 import com.intellij.refactoring.ui.ClassNameReferenceEditor;
 import com.intellij.refactoring.ui.PackageNameReferenceEditorCombo;
-import com.intellij.refactoring.ui.RefactoringDialog;
 import com.intellij.refactoring.util.CommonRefactoringUtil;
 import com.intellij.ui.ComboboxWithBrowseButton;
 import com.intellij.ui.RecentsManager;
@@ -337,24 +336,11 @@ public class MoveClassesOrPackagesDialog extends MoveDialogBase {
     }
     else {
       if (findTargetClass() == null) throw new ConfigurationException("Destination class not found");
-      final String validationError = getValidationError();
+      final String validationError = verifyInnerClassDestination();
       if (validationError != null) throw new ConfigurationException(validationError);
     }
   }
 
-  protected void validateButtons() {
-    super.validateButtons();
-    setErrorText(getValidationError());
-  }
-
-  @Nullable
-  private String getValidationError() {
-    if (!isMoveToPackage()) {
-      return verifyInnerClassDestination();
-    }
-    return null;
-  }
-
   @Nullable
   private PsiClass findTargetClass() {
     String name = myInnerClassChooser.getText().trim();
@@ -483,6 +469,7 @@ public class MoveClassesOrPackagesDialog extends MoveDialogBase {
   private void invokeMoveToInner() {
     saveRefactoringSettings();
     final PsiClass targetClass = findTargetClass();
+    if (targetClass == null) return;
     final PsiClass[] classesToMove = new PsiClass[myElementsToMove.length];
     for (int i = 0; i < myElementsToMove.length; i++) {
       classesToMove[i] = (PsiClass)myElementsToMove[i];
@@ -493,7 +480,7 @@ public class MoveClassesOrPackagesDialog extends MoveDialogBase {
   }
 
   //for scala plugin
-  protected MoveClassToInnerProcessor createMoveToInnerProcessor(PsiClass destination, @NotNull PsiClass[] classesToMove, @Nullable final MoveCallback callback) {
+  protected MoveClassToInnerProcessor createMoveToInnerProcessor(@NotNull PsiClass destination, @NotNull PsiClass[] classesToMove, @Nullable final MoveCallback callback) {
     return new MoveClassToInnerProcessor(getProject(), classesToMove, destination, isSearchInComments(), isSearchInNonJavaFiles(), callback);
   }