cleanup — ModuleListener
authorVladimir Krivosheev <vladimir.krivosheev@jetbrains.com>
Fri, 16 Dec 2016 10:09:02 +0000 (11:09 +0100)
committerVladimir Krivosheev <vladimir.krivosheev@jetbrains.com>
Sun, 18 Dec 2016 17:17:00 +0000 (18:17 +0100)
16 files changed:
java/compiler/impl/src/com/intellij/compiler/CompilerConfigurationImpl.java
java/java-tests/testSrc/com/intellij/roots/MultiModuleEditingTest.java
platform/configuration-store-impl/testSrc/ModuleStoreRenameTest.kt
platform/lang-impl/src/com/intellij/execution/impl/ModuleRunConfigurationManager.java
platform/lang-impl/src/com/intellij/facet/impl/ProjectWideFacetListenersRegistryImpl.java
platform/lang-impl/src/com/intellij/facet/impl/pointers/FacetPointersManagerImpl.java
platform/projectModel-api/src/com/intellij/openapi/project/ModuleAdapter.java
platform/projectModel-api/src/com/intellij/openapi/project/ModuleListener.java
platform/projectModel-impl/src/com/intellij/openapi/module/impl/ModulePointerManagerImpl.java
platform/testFramework/src/com/intellij/testFramework/LightPlatformTestCase.java
platform/vcs-impl/src/com/intellij/openapi/vcs/impl/ModuleVcsDetector.java
plugins/git4idea/remote-servers-git-java/src/com/intellij/remoteServer/util/importProject/CloudGitChooseAccountStepImpl.java
plugins/gradle/src/org/jetbrains/plugins/gradle/config/GradleResourceCompilerConfigurationGenerator.java
plugins/groovy/src/org/jetbrains/plugins/groovy/mvc/MvcModuleStructureSynchronizer.java
plugins/maven/src/main/java/org/jetbrains/idea/maven/project/MavenProjectsManagerWatcher.java
python/ide/src/com/jetbrains/python/PyModuleListener.java

index 09ab3995dc41f0df1b46c20c6208c999fb1db660..7815ab2d4d6eda273cf2b7c762efce7d6bf5e6e5 100644 (file)
@@ -38,7 +38,7 @@ import com.intellij.openapi.extensions.Extensions;
 import com.intellij.openapi.fileTypes.FileType;
 import com.intellij.openapi.module.Module;
 import com.intellij.openapi.module.ModuleManager;
-import com.intellij.openapi.project.ModuleAdapter;
+import com.intellij.openapi.project.ModuleListener;
 import com.intellij.openapi.project.Project;
 import com.intellij.openapi.roots.ProjectRootManager;
 import com.intellij.openapi.ui.InputValidator;
@@ -114,7 +114,7 @@ public class CompilerConfigurationImpl extends CompilerConfiguration implements
     myExcludesConfiguration = new ExcludesConfigNotificationsWrapper<>(project, new ExcludedEntriesConfiguration());
     Disposer.register(project, myExcludesConfiguration.getDelegate());
     MessageBusConnection connection = project.getMessageBus().connect(project);
-    connection.subscribe(ProjectTopics.MODULES, new ModuleAdapter() {
+    connection.subscribe(ProjectTopics.MODULES, new ModuleListener() {
       @Override
       public void beforeModuleRemoved(@NotNull Project project, @NotNull Module module) {
         getAnnotationProcessingConfiguration(module).removeModuleName(module.getName());
index 4cb6f86dfd037d512b2741533b40ecab171c0a1b..3d68da397085a964c66a37bac550497e23a96473 100644 (file)
@@ -23,7 +23,7 @@ import com.intellij.openapi.module.ModifiableModuleModel;
 import com.intellij.openapi.module.Module;
 import com.intellij.openapi.module.ModuleManager;
 import com.intellij.openapi.module.StdModuleTypes;
-import com.intellij.openapi.project.ModuleAdapter;
+import com.intellij.openapi.project.ModuleListener;
 import com.intellij.openapi.project.Project;
 import com.intellij.openapi.roots.ContentEntry;
 import com.intellij.openapi.roots.ModifiableRootModel;
@@ -189,7 +189,7 @@ public class MultiModuleEditingTest extends ModuleTestCase {
   }
 
 
-  private static class MyModuleListener extends ModuleAdapter {
+  private static class MyModuleListener implements ModuleListener {
     private final List<String> myLog = new ArrayList<>();
 
     @Override
index 105cfb8e35ad224d774d81aa6fffb927e2a1d562..cf0797c25888fbacdd39e1238fe48f6196bcb9a0 100644 (file)
@@ -7,7 +7,7 @@ import com.intellij.openapi.application.runWriteAction
 import com.intellij.openapi.components.StoragePathMacros
 import com.intellij.openapi.components.stateStore
 import com.intellij.openapi.module.Module
-import com.intellij.openapi.project.ModuleAdapter
+import com.intellij.openapi.project.ModuleListener
 import com.intellij.openapi.project.Project
 import com.intellij.openapi.vfs.LocalFileSystem
 import com.intellij.project.modifyModules
@@ -49,7 +49,7 @@ internal class ModuleStoreRenameTest {
           module = projectRule.createModule(tempDirManager.newPath(refreshVfs = true).resolve("m.iml"))
         }
 
-        module.messageBus.connect().subscribe(ProjectTopics.MODULES, object : ModuleAdapter() {
+        module.messageBus.connect().subscribe(ProjectTopics.MODULES, object : ModuleListener {
           override fun modulesRenamed(project: Project, modules: MutableList<Module>, oldNameProvider: Function<Module, String>) {
             assertThat(modules).containsOnly(module)
             oldModuleNames.add(oldNameProvider.`fun`(module))
index 77680aec1db4bf5f9ac9afe0515c2b43c2d97501..c0c01209af7f9c246a177bab4d3f8d3df22ab81d 100644 (file)
@@ -24,7 +24,7 @@ import com.intellij.openapi.components.State;
 import com.intellij.openapi.diagnostic.Logger;
 import com.intellij.openapi.module.Module;
 import com.intellij.openapi.module.ModuleComponent;
-import com.intellij.openapi.project.ModuleAdapter;
+import com.intellij.openapi.project.ModuleListener;
 import com.intellij.openapi.project.Project;
 import com.intellij.openapi.util.Comparing;
 import com.intellij.openapi.util.Condition;
@@ -43,7 +43,7 @@ import java.util.List;
 import java.util.Set;
 
 @State(name = ModuleRunConfigurationManager.COMPONENT_NAME)
-public final class ModuleRunConfigurationManager extends ModuleAdapter implements ModuleComponent, PersistentStateComponent<Element> {
+public final class ModuleRunConfigurationManager implements ModuleComponent, ModuleListener, PersistentStateComponent<Element> {
   private static final Logger LOG = Logger.getInstance(ModuleRunConfigurationManager.class);
   @NonNls static final String COMPONENT_NAME = "ModuleRunConfigurationManager";
   @NotNull
index d0c3e1407e6ec592c7054d8769ea71e726231ddf..ae0b0cc5c6add80f5294b74e36c3f4706415ebf0 100644 (file)
@@ -20,7 +20,7 @@ import com.intellij.ProjectTopics;
 import com.intellij.facet.*;
 import com.intellij.openapi.Disposable;
 import com.intellij.openapi.module.Module;
-import com.intellij.openapi.project.ModuleAdapter;
+import com.intellij.openapi.project.ModuleListener;
 import com.intellij.openapi.project.Project;
 import com.intellij.openapi.util.Disposer;
 import com.intellij.util.EventDispatcher;
@@ -44,7 +44,7 @@ public class ProjectWideFacetListenersRegistryImpl extends ProjectWideFacetListe
 
   public ProjectWideFacetListenersRegistryImpl(MessageBus messageBus) {
     myFacetListener = new MyFacetManagerAdapter();
-    messageBus.connect().subscribe(ProjectTopics.MODULES, new ModuleAdapter() {
+    messageBus.connect().subscribe(ProjectTopics.MODULES, new ModuleListener() {
       @Override
       public void moduleAdded(@NotNull Project project, @NotNull Module module) {
         onModuleAdded(module);
index aa7742d750d27700bcfbbc71bcef75d013da20f7..a8caed31a64113572ac1d7bed9aae050aa8c0a81 100644 (file)
@@ -28,7 +28,7 @@ import com.intellij.openapi.Disposable;
 import com.intellij.openapi.components.ProjectComponent;
 import com.intellij.openapi.module.Module;
 import com.intellij.openapi.module.ModuleManager;
-import com.intellij.openapi.project.ModuleAdapter;
+import com.intellij.openapi.project.ModuleListener;
 import com.intellij.openapi.project.Project;
 import com.intellij.openapi.util.Disposer;
 import com.intellij.openapi.util.Pair;
@@ -95,7 +95,7 @@ public class FacetPointersManagerImpl extends FacetPointersManager implements Pr
   @Override
   public void initComponent() {
     MessageBusConnection connection = myProject.getMessageBus().connect();
-    connection.subscribe(ProjectTopics.MODULES, new ModuleAdapter() {
+    connection.subscribe(ProjectTopics.MODULES, new ModuleListener() {
       @Override
       public void moduleAdded(@NotNull Project project, @NotNull Module module) {
         refreshPointers(module);
index f293b3dc92f4cd2a7e887993f29054d36d766f28..e469cf3338fdc023d4de352600c9ed7b6263a41c 100644 (file)
  */
 package com.intellij.openapi.project;
 
-import com.intellij.openapi.module.Module;
-import com.intellij.util.Function;
-import org.jetbrains.annotations.NotNull;
-
-import java.util.List;
-
 /**
- * @author nik
+ * @deprecated Please use ModuleListener directly.
  */
 public abstract class ModuleAdapter implements ModuleListener {
-  @Override
-  public void moduleAdded(@NotNull Project project, @NotNull Module module) { }
-
-  @Override
-  public void beforeModuleRemoved(@NotNull Project project, @NotNull Module module) { }
-
-  @Override
-  public void moduleRemoved(@NotNull Project project, @NotNull Module module) { }
-
-  @Override
-  public void modulesRenamed(@NotNull Project project, @NotNull List<Module> modules, @NotNull Function<Module, String> oldNameProvider) { }
 }
index 78ebff886f4280002167dec8b19592cf8c7222bd..311c7635d72cdab8bed941695af1082fd075895b 100644 (file)
@@ -26,11 +26,15 @@ import java.util.List;
  * @author max
  */
 public interface ModuleListener extends EventListener {
-  void moduleAdded(@NotNull Project project, @NotNull Module module);
+  default void moduleAdded(@NotNull Project project, @NotNull Module module) {
+  }
 
-  void beforeModuleRemoved(@NotNull Project project, @NotNull Module module);
+  default void beforeModuleRemoved(@NotNull Project project, @NotNull Module module) {
+  }
 
-  void moduleRemoved(@NotNull Project project, @NotNull Module module);
+  default void moduleRemoved(@NotNull Project project, @NotNull Module module) {
+  }
 
-  void modulesRenamed(@NotNull Project project, @NotNull List<Module> modules, @NotNull Function<Module, String> oldNameProvider);
+  default void modulesRenamed(@NotNull Project project, @NotNull List<Module> modules, @NotNull Function<Module, String> oldNameProvider) {
+  }
 }
index b2507ea0860ca0e5201fe28d5de903137fc48eb0..0ce85e403320af36d70b1b20861c55a8b0deb99e 100644 (file)
@@ -21,7 +21,7 @@ import com.intellij.openapi.module.Module;
 import com.intellij.openapi.module.ModuleManager;
 import com.intellij.openapi.module.ModulePointer;
 import com.intellij.openapi.module.ModulePointerManager;
-import com.intellij.openapi.project.ModuleAdapter;
+import com.intellij.openapi.project.ModuleListener;
 import com.intellij.openapi.project.Project;
 import com.intellij.openapi.util.Disposer;
 import com.intellij.util.Function;
@@ -41,7 +41,7 @@ public class ModulePointerManagerImpl extends ModulePointerManager {
 
   public ModulePointerManagerImpl(Project project) {
     myProject = project;
-    project.getMessageBus().connect().subscribe(ProjectTopics.MODULES, new ModuleAdapter() {
+    project.getMessageBus().connect().subscribe(ProjectTopics.MODULES, new ModuleListener() {
       @Override
       public void beforeModuleRemoved(@NotNull Project project, @NotNull Module module) {
         unregisterPointer(module);
index 302b0628697304dc8e2d7ef6d05365384b36f5e8..eae62f7e653c4af648c043c04a1a7c4da21472f2 100644 (file)
@@ -52,7 +52,7 @@ import com.intellij.openapi.module.EmptyModuleType;
 import com.intellij.openapi.module.Module;
 import com.intellij.openapi.module.ModuleType;
 import com.intellij.openapi.project.DumbService;
-import com.intellij.openapi.project.ModuleAdapter;
+import com.intellij.openapi.project.ModuleListener;
 import com.intellij.openapi.project.Project;
 import com.intellij.openapi.project.ProjectManager;
 import com.intellij.openapi.project.ex.ProjectManagerEx;
@@ -305,7 +305,7 @@ public abstract class LightPlatformTestCase extends UsefulTestCase implements Da
     }
 
     MessageBusConnection connection = ourProject.getMessageBus().connect(parentDisposable);
-    connection.subscribe(ProjectTopics.MODULES, new ModuleAdapter() {
+    connection.subscribe(ProjectTopics.MODULES, new ModuleListener() {
       @Override
       public void moduleAdded(@NotNull Project project, @NotNull Module module) {
         fail("Adding modules is not permitted in LightIdeaTestCase.");
index 9a65d8d5debb26995a0264fcbfb4c949f02978f8..9fd464a1f6b7907a718c4b356860eaaaa838e5c6 100644 (file)
@@ -21,7 +21,7 @@ import com.intellij.openapi.application.ApplicationManager;
 import com.intellij.openapi.components.ProjectComponent;
 import com.intellij.openapi.module.Module;
 import com.intellij.openapi.module.ModuleManager;
-import com.intellij.openapi.project.ModuleAdapter;
+import com.intellij.openapi.project.ModuleListener;
 import com.intellij.openapi.project.Project;
 import com.intellij.openapi.roots.ModuleRootEvent;
 import com.intellij.openapi.roots.ModuleRootListener;
@@ -82,7 +82,7 @@ public class ModuleVcsDetector implements ProjectComponent {
     });
   }
 
-  private class MyModulesListener extends ModuleAdapter implements ModuleRootListener {
+  private class MyModulesListener implements ModuleRootListener, ModuleListener {
     private final List<Pair<String, VcsDirectoryMapping>> myMappingsForRemovedModules = new ArrayList<>();
 
     @Override
index ba5340e1c3628fb1cd5c1ddfc212c47c82f5cfd3..4e273c4fc9c51c23cbf503d51c6e14c062562c32 100644 (file)
@@ -24,7 +24,7 @@ import com.intellij.ide.util.projectWizard.importSources.DetectedSourceRoot;
 import com.intellij.ide.util.projectWizard.importSources.ProjectFromSourcesBuilder;
 import com.intellij.openapi.module.Module;
 import com.intellij.openapi.module.StdModuleTypes;
-import com.intellij.openapi.project.ModuleAdapter;
+import com.intellij.openapi.project.ModuleListener;
 import com.intellij.openapi.project.Project;
 import com.intellij.openapi.roots.ModifiableRootModel;
 import com.intellij.openapi.startup.StartupManager;
@@ -111,7 +111,7 @@ public class CloudGitChooseAccountStepImpl extends CloudGitChooseAccountStepBase
         @Override
         public void update(final @NotNull Module module, @NotNull ModifiableRootModel rootModel) {
           final MessageBusConnection connection = module.getProject().getMessageBus().connect();
-          connection.subscribe(ProjectTopics.MODULES, new ModuleAdapter() {
+          connection.subscribe(ProjectTopics.MODULES, new ModuleListener() {
 
             @Override
             public void moduleAdded(@NotNull Project project, @NotNull Module addedModule) {
index 00b8aab44712cb49fcc31b875d7bc6ccbb387b7c..14abae3cb9e563a65be2821a52b53a257285e603 100644 (file)
@@ -25,7 +25,7 @@ import com.intellij.openapi.diagnostic.Logger;
 import com.intellij.openapi.externalSystem.model.project.ExternalSystemSourceType;
 import com.intellij.openapi.externalSystem.util.ExternalSystemApiUtil;
 import com.intellij.openapi.module.Module;
-import com.intellij.openapi.project.ModuleAdapter;
+import com.intellij.openapi.project.ModuleListener;
 import com.intellij.openapi.project.Project;
 import com.intellij.openapi.project.ProjectType;
 import com.intellij.openapi.project.ProjectTypeService;
@@ -78,7 +78,7 @@ public class GradleResourceCompilerConfigurationGenerator {
     externalProjectDataCache = ExternalProjectDataCache.getInstance(project);
     assert externalProjectDataCache != null;
 
-    project.getMessageBus().connect(project).subscribe(ProjectTopics.MODULES, new ModuleAdapter() {
+    project.getMessageBus().connect(project).subscribe(ProjectTopics.MODULES, new ModuleListener() {
       @Override
       public void moduleRemoved(@NotNull Project project, @NotNull Module module) {
         myModulesConfigurationHash.remove(module.getName());
index a1f9b8563b3631247863890cc50801b9340851b4..229c6aadbbadb8c30befc43620f5351621362c5b 100644 (file)
@@ -29,7 +29,7 @@ import com.intellij.openapi.progress.ProgressIndicator;
 import com.intellij.openapi.progress.util.ProgressIndicatorUtils;
 import com.intellij.openapi.progress.util.ReadTask;
 import com.intellij.openapi.project.DumbAwareRunnable;
-import com.intellij.openapi.project.ModuleAdapter;
+import com.intellij.openapi.project.ModuleListener;
 import com.intellij.openapi.project.Project;
 import com.intellij.openapi.roots.*;
 import com.intellij.openapi.startup.StartupManager;
@@ -96,7 +96,7 @@ public class MvcModuleStructureSynchronizer extends AbstractProjectComponent {
       }
     });
 
-    connection.subscribe(ProjectTopics.MODULES, new ModuleAdapter() {
+    connection.subscribe(ProjectTopics.MODULES, new ModuleListener() {
       @Override
       public void moduleAdded(@NotNull Project project, @NotNull Module module) {
         queue(SyncAction.UpdateProjectStructure, module);
index 122bad170bfc11edff80f69a7adda52fdf97ba6d..f5c73ea847915c9864edfe5a39cb934615cc37a3 100644 (file)
@@ -26,7 +26,7 @@ import com.intellij.openapi.editor.event.DocumentEvent;
 import com.intellij.openapi.fileEditor.FileDocumentManager;
 import com.intellij.openapi.fileEditor.impl.FileDocumentManagerImpl;
 import com.intellij.openapi.module.Module;
-import com.intellij.openapi.project.ModuleAdapter;
+import com.intellij.openapi.project.ModuleListener;
 import com.intellij.openapi.project.Project;
 import com.intellij.openapi.roots.ModuleRootEvent;
 import com.intellij.openapi.roots.ModuleRootListener;
@@ -115,7 +115,7 @@ public class MavenProjectsManagerWatcher {
     myChangedDocumentsQueue.makeUserAware(myProject);
     myChangedDocumentsQueue.activate();
 
-    myBusConnection.subscribe(ProjectTopics.MODULES, new ModuleAdapter() {
+    myBusConnection.subscribe(ProjectTopics.MODULES, new ModuleListener() {
       @Override
       public void moduleRemoved(@NotNull Project project, @NotNull Module module) {
         MavenProject mavenProject = myManager.findProject(module);
index 31ffc87fa0569161453263f98ea9d44db4b9db03..2646174ac178757f34a8ff1a52bd43cb94134315 100644 (file)
@@ -19,7 +19,7 @@ import com.intellij.ProjectTopics;
 import com.intellij.execution.RunManagerEx;
 import com.intellij.execution.RunnerAndConfigurationSettings;
 import com.intellij.openapi.module.Module;
-import com.intellij.openapi.project.ModuleAdapter;
+import com.intellij.openapi.project.ModuleListener;
 import com.intellij.openapi.project.Project;
 import com.intellij.util.messages.MessageBus;
 import com.jetbrains.python.run.AbstractPythonRunConfiguration;
@@ -33,7 +33,7 @@ import java.util.Collection;
  */
 public class PyModuleListener {
   public PyModuleListener(MessageBus messageBus) {
-    messageBus.connect().subscribe(ProjectTopics.MODULES, new ModuleAdapter() {
+    messageBus.connect().subscribe(ProjectTopics.MODULES, new ModuleListener() {
       @Override
       public void beforeModuleRemoved(@NotNull Project project, @NotNull Module module) {
         final RunManagerEx runManager = RunManagerEx.getInstanceEx(project);