diff --git a/platform/lang-impl/src/com/intellij/codeInsight/actions/OptimizeImportsAction.java b/platform/lang-impl/src/com/intellij/codeInsight/actions/OptimizeImportsAction.java index bc18a4241360..0295fee17973 100644 --- a/platform/lang-impl/src/com/intellij/codeInsight/actions/OptimizeImportsAction.java +++ b/platform/lang-impl/src/com/intellij/codeInsight/actions/OptimizeImportsAction.java @@ -59,7 +59,7 @@ public class OptimizeImportsAction extends AnAction { if (file == null) return; dir = file.getContainingDirectory(); } - else if (files != null && ReformatCodeAction.areFiles(files)) { + else if (files != null && ReformatCodeAction.containsAtLeastOneFile(files)) { final ReadonlyStatusHandler.OperationStatus operationStatus = ReadonlyStatusHandler.getInstance(project).ensureFilesWritable(files); if (!operationStatus.hasReadonlyFiles()) { new OptimizeImportsProcessor(project, ReformatCodeAction.convertToPsiFiles(files, project), null).run(); @@ -160,7 +160,7 @@ public class OptimizeImportsAction extends AnAction { return; } } - else if (files != null && ReformatCodeAction.areFiles(files)) { + else if (files != null && ReformatCodeAction.containsAtLeastOneFile(files)) { boolean anyHasOptimizeImports = false; for (VirtualFile virtualFile : files) { PsiFile file = PsiManager.getInstance(project).findFile(virtualFile); diff --git a/platform/lang-impl/src/com/intellij/codeInsight/actions/ReformatCodeAction.java b/platform/lang-impl/src/com/intellij/codeInsight/actions/ReformatCodeAction.java index 78c6b9a6cff6..ba3d2d961f00 100644 --- a/platform/lang-impl/src/com/intellij/codeInsight/actions/ReformatCodeAction.java +++ b/platform/lang-impl/src/com/intellij/codeInsight/actions/ReformatCodeAction.java @@ -88,7 +88,7 @@ public class ReformatCodeAction extends AnAction implements DumbAware { dir = file.getContainingDirectory(); hasSelection = editor.getSelectionModel().hasSelection(); } - else if (areFiles(files)) { + else if (containsAtLeastOneFile(files)) { final ReadonlyStatusHandler.OperationStatus operationStatus = ReadonlyStatusHandler.getInstance(project).ensureFilesWritable(files); if (!operationStatus.hasReadonlyFiles()) { ReformatFilesOptions selectedFlags = getReformatFilesOptions(project, files); @@ -118,9 +118,6 @@ public class ReformatCodeAction extends AnAction implements DumbAware { } return; } - else if (files != null && files.length == 1) { - file = PsiManager.getInstance(project).findFile(files[0]); - } else { PsiElement element = CommonDataKeys.PSI_ELEMENT.getData(dataContext); if (element == null) return; @@ -385,7 +382,7 @@ public class ReformatCodeAction extends AnAction implements DumbAware { return; } } - else if (files!= null && areFiles(files)) { + else if (files!= null && containsAtLeastOneFile(files)) { boolean anyFormatters = false; for (VirtualFile virtualFile : files) { if (virtualFile.isDirectory()) { @@ -490,9 +487,9 @@ public class ReformatCodeAction extends AnAction implements DumbAware { myTestOptions = options; } - public static boolean areFiles(final VirtualFile[] files) { + public static boolean containsAtLeastOneFile(final VirtualFile[] files) { if (files == null) return false; - if (files.length < 2) return false; + if (files.length < 1) return false; for (VirtualFile virtualFile : files) { if (virtualFile.isDirectory()) return false; }