From af82fb8e9566767fa9b3034b2a585c10638db0d0 Mon Sep 17 00:00:00 2001 From: Anna Kozlova Date: Fri, 16 Jun 2017 13:46:22 +0300 Subject: [PATCH] null checks (IDEA-CR-21943) --- .../MoveDirectoryWithClassesProcessor.java | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/platform/lang-impl/src/com/intellij/refactoring/move/moveClassesOrPackages/MoveDirectoryWithClassesProcessor.java b/platform/lang-impl/src/com/intellij/refactoring/move/moveClassesOrPackages/MoveDirectoryWithClassesProcessor.java index 0992907dd463..12d6799679df 100644 --- a/platform/lang-impl/src/com/intellij/refactoring/move/moveClassesOrPackages/MoveDirectoryWithClassesProcessor.java +++ b/platform/lang-impl/src/com/intellij/refactoring/move/moveClassesOrPackages/MoveDirectoryWithClassesProcessor.java @@ -102,7 +102,7 @@ public class MoveDirectoryWithClassesProcessor extends BaseRefactoringProcessor } private Set getPsiFiles() { - return myFilesToMove.keySet().stream().map(myManager::findFile).collect(Collectors.toSet()); + return myFilesToMove.keySet().stream().map(myManager::findFile).filter(Objects::nonNull).collect(Collectors.toSet()); } protected String getTargetName() { @@ -124,6 +124,7 @@ public class MoveDirectoryWithClassesProcessor extends BaseRefactoringProcessor final MultiMap conflicts = new MultiMap<>(); for (VirtualFile vFile : myFilesToMove.keySet()) { PsiFile file = myManager.findFile(vFile); + if (file == null) continue; try { myFilesToMove.get(vFile).checkMove(file); } @@ -163,6 +164,7 @@ public class MoveDirectoryWithClassesProcessor extends BaseRefactoringProcessor final Map oldToNewElementsMapping = new HashMap<>(); for (VirtualFile virtualFile : myFilesToMove.keySet()) { PsiFile file = myManager.findFile(virtualFile); + if (file == null) continue; for (MoveDirectoryWithClassesHelper helper : MoveDirectoryWithClassesHelper.findAll()) { helper.beforeMove(file); }