ProjectFileIndex#isIgnored renamed to 'isExcluded'
authornik <Nikolay.Chashnikov@jetbrains.com>
Tue, 22 Jul 2014 12:30:56 +0000 (16:30 +0400)
committernik <Nikolay.Chashnikov@jetbrains.com>
Tue, 22 Jul 2014 12:31:23 +0000 (16:31 +0400)
20 files changed:
images/src/org/intellij/images/thumbnail/impl/ThumbnailViewUI.java
java/java-impl/src/com/intellij/psi/impl/JavaDirectoryIconProvider.java
java/java-tests/testSrc/com/intellij/openapi/roots/impl/DirectoryIndexTest.java
platform/analysis-api/src/com/intellij/analysis/AnalysisScope.java
platform/analysis-api/src/com/intellij/psi/search/scope/ProjectFilesScope.java
platform/lang-impl/src/com/intellij/ide/projectView/impl/nodes/ProjectViewDirectoryHelper.java
platform/lang-impl/src/com/intellij/ide/projectView/impl/nodes/PsiDirectoryNode.java
platform/lang-impl/src/com/intellij/psi/impl/file/DirectoryIconProvider.java
platform/lang-impl/src/com/intellij/psi/impl/file/impl/PsiVFSListener.java
platform/lvcs-impl/src/com/intellij/history/integration/IdeaGateway.java
platform/projectModel-api/src/com/intellij/openapi/roots/ProjectFileIndex.java
platform/projectModel-impl/src/com/intellij/openapi/roots/impl/ProjectFileIndexFacade.java
platform/projectModel-impl/src/com/intellij/openapi/roots/impl/ProjectFileIndexImpl.java
plugins/cvs/cvs-plugin/src/com/intellij/cvsSupport2/cvsstatuses/CvsChangeProvider.java
plugins/cvs/cvs-plugin/src/com/intellij/cvsSupport2/ui/experts/importToCvs/ImportTree.java
plugins/eclipse/src/org/jetbrains/idea/eclipse/conversion/EPathUtil.java
plugins/eclipse/src/org/jetbrains/idea/eclipse/conversion/IdeaSpecificSettings.java
plugins/groovy/src/org/jetbrains/plugins/groovy/compiler/GroovyCompilerConfigurable.java
plugins/groovy/src/org/jetbrains/plugins/groovy/mvc/MvcModuleStructureUtil.java
python/src/com/jetbrains/python/packaging/PyPackageUtil.java

index cef0e6cb7184d879da00266e63d0e9a723bc1666..83f7679fcc62993036fe8a49cfd8878a62a457a1 100644 (file)
@@ -300,7 +300,7 @@ final class ThumbnailViewUI extends JPanel implements DataProvider, Disposable {
         Project project = thumbnailView.getProject();
         if (!project.isDisposed()) {
             ProjectRootManager rootManager = ProjectRootManager.getInstance(project);
-            boolean projectIgnored = rootManager.getFileIndex().isIgnored(file);
+            boolean projectIgnored = rootManager.getFileIndex().isExcluded(file);
 
             if (!projectIgnored && !FileTypeManager.getInstance().isFileIgnored(file)) {
                 ImageFileTypeManager typeManager = ImageFileTypeManager.getInstance();
index f426e913dde3318e927fa9884ade840fc7639a25..e87e95f79086ed8815fdb9e63bd42bbb8c4228fd 100644 (file)
@@ -70,7 +70,7 @@ public class JavaDirectoryIconProvider extends IconProvider implements DumbAware
       else if (JavaDirectoryService.getInstance().getPackage(psiDirectory) != null) {
         symbolIcon = PlatformIcons.PACKAGE_ICON;
       }
-      else if (!Registry.is("ide.hide.excluded.files") && ProjectRootManager.getInstance(project).getFileIndex().isIgnored(vFile)) {
+      else if (!Registry.is("ide.hide.excluded.files") && ProjectRootManager.getInstance(project).getFileIndex().isExcluded(vFile)) {
         symbolIcon = AllIcons.Modules.ExcludeRoot;
       }
       else {
index e583daed6ed1c43960471048c1c36551859bf4c7..1e3e08a33ae7b5f1c44a90015c1ed1259d9fad6c 100644 (file)
@@ -362,7 +362,7 @@ public class DirectoryIndexTest extends IdeaTestCase {
     VirtualFile ignoredFile = myModule1Dir.createChildData(this, "CVS");
     DirectoryInfo info = myIndex.getInfoForFile(ignoredFile);
     assertTrue(info.isIgnored());
-    assertTrue(ProjectRootManager.getInstance(myProject).getFileIndex().isIgnored(ignoredFile));
+    assertTrue(ProjectRootManager.getInstance(myProject).getFileIndex().isExcluded(ignoredFile));
   }
 
   public void testAddModule() throws Exception {
@@ -386,10 +386,10 @@ public class DirectoryIndexTest extends IdeaTestCase {
   public void testModuleUnderIgnoredDir() throws IOException {
     final VirtualFile ignored = myRootVFile.createChildDirectory(this, "RCS");
     assertTrue(FileTypeManager.getInstance().isFileIgnored(ignored));
-    assertTrue(ProjectRootManager.getInstance(myProject).getFileIndex().isIgnored(ignored));
+    assertTrue(ProjectRootManager.getInstance(myProject).getFileIndex().isExcluded(ignored));
     final VirtualFile module4 = ignored.createChildDirectory(this, "module4");
     assertFalse(FileTypeManager.getInstance().isFileIgnored(module4));
-    assertTrue(ProjectRootManager.getInstance(myProject).getFileIndex().isIgnored(module4));
+    assertTrue(ProjectRootManager.getInstance(myProject).getFileIndex().isExcluded(module4));
 
     new WriteCommandAction.Simple(getProject()) {
       @Override
@@ -427,9 +427,9 @@ public class DirectoryIndexTest extends IdeaTestCase {
   public void testExcludedDirsInLibraries() {
     ProjectFileIndex index = ProjectRootManager.getInstance(myProject).getFileIndex();
     assertFalse(index.isInLibraryClasses(myExcludedLibClsDir));
-    assertTrue(index.isIgnored(myExcludedLibClsDir));
+    assertTrue(index.isExcluded(myExcludedLibClsDir));
     assertFalse(index.isInLibrarySource(myExcludedLibSrcDir));
-    assertTrue(index.isIgnored(myExcludedLibSrcDir));
+    assertTrue(index.isExcluded(myExcludedLibSrcDir));
   }
 
   public void testExplicitExcludeOfInner() throws Exception {
@@ -662,9 +662,9 @@ public class DirectoryIndexTest extends IdeaTestCase {
 
   public void testExcludeCompilerOutputOutsideOfContentRoot() throws Exception {
     final ProjectFileIndex fileIndex = ProjectRootManager.getInstance(myProject).getFileIndex();
-    assertTrue(fileIndex.isIgnored(myOutputDir));
-    assertTrue(fileIndex.isIgnored(myModule1OutputDir));
-    assertFalse(fileIndex.isIgnored(myOutputDir.getParent()));
+    assertTrue(fileIndex.isExcluded(myOutputDir));
+    assertTrue(fileIndex.isExcluded(myModule1OutputDir));
+    assertFalse(fileIndex.isExcluded(myOutputDir.getParent()));
     assertExcludedFromProject(myOutputDir);
     assertExcludedFromProject(myModule1OutputDir);
     String moduleOutputUrl = myModule1OutputDir.getUrl();
@@ -677,7 +677,7 @@ public class DirectoryIndexTest extends IdeaTestCase {
 
     assertExcludedFromProject(myOutputDir);
     assertExcludedFromProject(myModule1OutputDir);
-    assertTrue(fileIndex.isIgnored(myModule1OutputDir));
+    assertTrue(fileIndex.isExcluded(myModule1OutputDir));
 
     PsiTestUtil.setCompilerOutputPath(myModule, moduleOutputUrl, true);
     PsiTestUtil.setCompilerOutputPath(myModule2, moduleOutputUrl, false);
index f1255edc01f617a60f66c9acdc23a6ab5be79ff3..b18bb042c183a0bbe5c6daee4bfca2aa990049ec 100644 (file)
@@ -249,7 +249,7 @@ public class AnalysisScope {
           @NotNull
           @Override
           public Result visitFileEx(@NotNull VirtualFile file) {
-            boolean ignored = fileIndex.isIgnored(file);
+            boolean ignored = fileIndex.isExcluded(file);
             if (!ignored && !file.isDirectory()) {
               files.add(file);
             }
index 52313eaaceab44d0d03f7eb8ac094de1213e5e5b..e55cd9d5cfbddbca718fc2dc2acbf6a1e9f4e8c4 100644 (file)
@@ -42,7 +42,7 @@ public class ProjectFilesScope extends NamedScope {
         if (file == null) return false;
         final ProjectFileIndex fileIndex = ProjectRootManager.getInstance(project).getFileIndex();
         return holder.getProject().isInitialized()
-               && !fileIndex.isIgnored(file)
+               && !fileIndex.isExcluded(file)
                && fileIndex.getContentRootForFile(file) != null;
       }
     });
index 02db52a270b38204134959a8f99001a75df110eb..13ba919005215a9a06b876d1314b6b0c66df6436 100644 (file)
@@ -142,7 +142,7 @@ public class ProjectViewDirectoryHelper {
         VirtualFile directoryFile = subdir.getVirtualFile();
 
         if (Registry.is("ide.hide.excluded.files")) {
-          if (fileIndex.isIgnored(directoryFile)) continue;
+          if (fileIndex.isExcluded(directoryFile)) continue;
         }
         else {
           if (FileTypeRegistry.getInstance().isFileIgnored(directoryFile)) continue;
index c0a75fccf781c7eff0aac8f6e1be5f0ec114bad2..16aa930b806a81fab77646e84cbdf05abfe88a2b 100644 (file)
@@ -188,7 +188,7 @@ public class PsiDirectoryNode extends BasePsiNode<PsiDirectory> implements Navig
     if (Registry.is("ide.hide.excluded.files")) {
       final Project project = value.getProject();
       final ProjectFileIndex fileIndex = ProjectRootManager.getInstance(project).getFileIndex();
-      return !fileIndex.isIgnored(file);
+      return !fileIndex.isExcluded(file);
     }
     else {
       return !FileTypeRegistry.getInstance().isFileIgnored(file);
index 31cd6c2c43f53233877af4924b1830ad73e2651e..6aaa810340bd76ddd902969fdc9a9b894f176fb9 100644 (file)
@@ -50,7 +50,7 @@ public class DirectoryIconProvider extends IconProvider implements DumbAware {
       }
       else {
         if (!Registry.is("ide.hide.excluded.files")) {
-          boolean ignored = ProjectRootManager.getInstance(project).getFileIndex().isIgnored(vFile);
+          boolean ignored = ProjectRootManager.getInstance(project).getFileIndex().isExcluded(vFile);
           if (ignored) {
             return AllIcons.Modules.ExcludeRoot;
           }
index bb2c938e3d0884ee0cec35828337d6e9b57e83a5..17dd2bca1f17681073822d8996196c2fc0a9e2a9 100644 (file)
@@ -443,7 +443,7 @@ public class PsiVFSListener extends VirtualFileAdapter {
         public void run() {
           PsiTreeChangeEventImpl treeEvent = new PsiTreeChangeEventImpl(myManager);
 
-          boolean isExcluded = vFile.isDirectory() && myProjectRootManager.getFileIndex().isIgnored(vFile);
+          boolean isExcluded = vFile.isDirectory() && myProjectRootManager.getFileIndex().isExcluded(vFile);
           if (oldParentDir != null && !isExcluded) {
             if (newParentDir != null) {
               treeEvent.setOldParent(oldParentDir);
index 1a916917d038a31a34221802b1fc89eed30ff385..30822c0aec7943a4b53c2165c1362aee78001878 100644 (file)
@@ -73,7 +73,7 @@ public class IdeaGateway {
       if (Comparing.equal(each.getWorkspaceFile(), f)) return false;
       ProjectFileIndex index = ProjectRootManager.getInstance(each).getFileIndex();
       
-      if (index.isIgnored(f)) return false;
+      if (index.isExcluded(f)) return false;
       isInContent |= index.isInContent(f);
     }
     if (shouldBeInContent && !isInContent) return false;
index 8ada83981017aff61ad0090ab56777d77f182514..b06cf065fd9518f7a7118e2a692e0eb057792a2c 100644 (file)
@@ -129,11 +129,18 @@ public interface ProjectFileIndex extends FileIndex {
   boolean isInLibrarySource(@NotNull VirtualFile fileOrDir);
 
   /**
-   * Checks if the specified file or directory is ignored (either excluded by exclude roots
-   * or ignored by {@link com.intellij.openapi.fileTypes.FileTypeManager#isFileIgnored(String)}).
+   * @deprecated name of this method may be confusing. If you want to check if the file is excluded or ignored use {@link #isExcluded(com.intellij.openapi.vfs.VirtualFile)}.
+   * If you want to check if the file is ignored use {@link com.intellij.openapi.fileTypes.FileTypeRegistry#isFileIgnored(com.intellij.openapi.vfs.VirtualFile)}.
+   */
+  @Deprecated
+  boolean isIgnored(@NotNull VirtualFile file);
+
+  /**
+   * Checks if the specified file or directory is located under project roots but the file itself or one of its parent directories is
+   * either excluded from the project or ignored by {@link com.intellij.openapi.fileTypes.FileTypeRegistry#isFileIgnored(com.intellij.openapi.vfs.VirtualFile)}).
    *
    * @param file the file to check.
-   * @return true if <code>file</code> is ignored, false otherwise.
+   * @return true if <code>file</code> is excluded or ignored, false otherwise.
    */
-  boolean isIgnored(@NotNull VirtualFile file);
+  boolean isExcluded(@NotNull VirtualFile file);
 }
index 5a4aecde8018a5144d594635512be69be516b8e1..0bb65c60008311c14f3bbc75c36c4b552b42f292 100644 (file)
@@ -66,7 +66,7 @@ public class ProjectFileIndexFacade extends FileIndexFacade {
 
   @Override
   public boolean isExcludedFile(@NotNull final VirtualFile file) {
-    return myFileIndex.isIgnored(file);
+    return myFileIndex.isExcluded(file);
   }
 
   @Nullable
index 839ce38e540407c32c1cc476d551923f0a755b41..acb6598f55c8915ca88c091980b655c089e6952c 100644 (file)
@@ -79,7 +79,7 @@ public class ProjectFileIndexImpl extends FileIndexBase implements ProjectFileIn
   }
 
   @Override
-  public boolean isIgnored(@NotNull VirtualFile file) {
+  public boolean isExcluded(@NotNull VirtualFile file) {
     DirectoryInfo info = getInfoForFileOrDirectory(file);
     return info.isIgnored() || info.isExcluded();
   }
@@ -150,6 +150,11 @@ public class ProjectFileIndexImpl extends FileIndexBase implements ProjectFileIn
   }
 
   @Override
+  public boolean isIgnored(@NotNull VirtualFile file) {
+    return isExcluded(file);
+  }
+
+  @Override
   public boolean isInContent(@NotNull VirtualFile fileOrDir) {
     DirectoryInfo info = getInfoForFileOrDirectory(fileOrDir);
     return info.isInProject() && info.getModule() != null;
index ee82d08745669c76fa10e6354e0f1790def43957..a2e055b7b283dec95f828d3c156ee377b871fe1a 100644 (file)
@@ -169,7 +169,7 @@ public class CvsChangeProvider implements ChangeProvider {
         for (VirtualFile file : children) {
           progress.checkCanceled();
           if (file.isDirectory()) {
-            final boolean isIgnored = myFileIndex.isIgnored(file);
+            final boolean isIgnored = myFileIndex.isExcluded(file);
             if (!isIgnored) {
               processEntriesIn(file, scope, builder, true, progress);
             }
index 99f038430b9b8dbcc51f67a806a35114f8db25dd..3877a78a0d353d35bb365866efb26f6d81647db5 100644 (file)
@@ -151,11 +151,11 @@ public class ImportTree extends NodeRenderer {
       return;
     }
     final ProjectFileIndex fileIndex = ProjectRootManager.getInstance(myProject).getFileIndex();
-    if (!fileIndex.isIgnored(selectedFile)) {
+    if (!fileIndex.isExcluded(selectedFile)) {
       return;
     }
     final VirtualFile parent = selectedFile.getParent();
-    if (parent != null && fileIndex.isIgnored(parent)) {
+    if (parent != null && fileIndex.isExcluded(parent)) {
       return;
     }
     for (final VirtualFile excludedFile : myExcludedFiles) {
@@ -183,11 +183,11 @@ public class ImportTree extends NodeRenderer {
         continue;
       }
       final ProjectFileIndex fileIndex = ProjectRootManager.getInstance(myProject).getFileIndex();
-      if (!fileIndex.isIgnored(selectedFile)) {
+      if (!fileIndex.isExcluded(selectedFile)) {
         continue;
       }
       final VirtualFile parent = selectedFile.getParent();
-      if (parent == null || fileIndex.isIgnored(parent) || myExcludedFiles.contains(parent)) {
+      if (parent == null || fileIndex.isExcluded(parent) || myExcludedFiles.contains(parent)) {
         continue;
       }
       if (!myIncludedFiles.contains(selectedFile)) {
@@ -211,7 +211,7 @@ public class ImportTree extends NodeRenderer {
         return true;
       }
     }
-    if (myProject == null || !ProjectRootManager.getInstance(myProject).getFileIndex().isIgnored(file)) {
+    if (myProject == null || !ProjectRootManager.getInstance(myProject).getFileIndex().isExcluded(file)) {
       return false;
     }
     for (VirtualFile includedFile : myIncludedFiles) {
@@ -242,7 +242,7 @@ public class ImportTree extends NodeRenderer {
         if (FileTypeManager.getInstance().isFileIgnored(abstractFileObject.getName())) return true;
         if (myProject != null && !includedFiles.contains(file)) {
           final VirtualFile vFile = LocalFileSystem.getInstance().findFileByIoFile(file);
-          if (vFile != null && ProjectRootManager.getInstance(myProject).getFileIndex().isIgnored(vFile)) {
+          if (vFile != null && ProjectRootManager.getInstance(myProject).getFileIndex().isExcluded(vFile)) {
             return true;
           }
         }
index c5a23e1b1d43c07a73dd3a87e8cef9fe0bed37d7..a29672091751b26c98ea8dda8c792b0e55570113 100644 (file)
@@ -149,7 +149,7 @@ public class EPathUtil {
       return collapse2eclipsePathRelative2Module(file, module);
     } else { //should check all modules then
       final ProjectFileIndex fileIndex = ProjectRootManager.getInstance(project).getFileIndex();
-      if (fileIndex.isIgnored(file) || fileIndex.isInLibraryClasses(file)) {
+      if (fileIndex.isExcluded(file) || fileIndex.isInLibraryClasses(file)) {
         for (Module aModule : ModuleManager.getInstance(project).getModules()) {
           final String path = collapse2eclipsePathRelative2Module(file, aModule);
           if (path != null) {
index ffd71fd6ef765bf4bb8e19fd6c8924cf0d31bb59..4541cf57f7a773a89102b2ad9d6c8dee3e185ca8 100644 (file)
@@ -451,7 +451,7 @@ public class IdeaSpecificSettings extends AbstractIdeaSpecificSettings<Modifiabl
       final Module module = ModuleUtilCore.findModuleForFile(file, project);
       if (module != null) {
         return appendRelatedToModule(element, classesUrl, rootName, file, module);
-      } else if (ProjectRootManager.getInstance(project).getFileIndex().isIgnored(file)) {
+      } else if (ProjectRootManager.getInstance(project).getFileIndex().isExcluded(file)) {
         for (Module aModule : ModuleManager.getInstance(project).getModules()) {
           if (appendRelatedToModule(element, classesUrl, rootName, file, aModule)) return true;
         }
index f273be1b3d6ef48f286182da4b6b95500d7c0462..2db660e1cf04fd343692abe826f53a23516e717a 100644 (file)
@@ -67,7 +67,7 @@ public class GroovyCompilerConfigurable implements SearchableConfigurable, Confi
     final FileChooserDescriptor descriptor = new FileChooserDescriptor(true, true, false, false, false, true) {
       @Override
       public boolean isFileVisible(VirtualFile file, boolean showHiddenFiles) {
-        return super.isFileVisible(file, showHiddenFiles) && !index.isIgnored(file);
+        return super.isFileVisible(file, showHiddenFiles) && !index.isExcluded(file);
       }
     };
     descriptor.setRoots(ContainerUtil.concat(
index df0e67385c8c9d5e8e2bb6f63e04c0af1d82e48e..2bfa261f420cc75c0770387460368eaac232f9a8 100644 (file)
@@ -169,7 +169,7 @@ public class MvcModuleStructureUtil {
   @Nullable
   public static Consumer<ModifiableRootModel> addJarDirectory(VirtualFile root, Module module, final String libName) {
     final VirtualFile libDir = root.findFileByRelativePath("lib");
-    if (libDir == null || !libDir.isDirectory() || ProjectRootManager.getInstance(module.getProject()).getFileIndex().isIgnored(libDir)) {
+    if (libDir == null || !libDir.isDirectory() || ProjectRootManager.getInstance(module.getProject()).getFileIndex().isExcluded(libDir)) {
       return null;
     }
 
index 3bfcc497cac93f8205b517ca9bdfdef611b04b7e..31258fa69ea5575798fcf1297f350f9838edf300 100644 (file)
@@ -181,7 +181,7 @@ public class PyPackageUtil {
     VfsUtilCore.visitChildrenRecursively(root, new VirtualFileVisitor() {
       @Override
       public boolean visitFile(@NotNull VirtualFile file) {
-        if (!fileIndex.isIgnored(file) && file.isDirectory() && file.findChild(PyNames.INIT_DOT_PY) != null) {
+        if (!fileIndex.isExcluded(file) && file.isDirectory() && file.findChild(PyNames.INIT_DOT_PY) != null) {
           results.add(VfsUtilCore.getRelativePath(file, root, '.'));
         }
         return true;