SONAR-926 No more call shouldExecuteOnProject on extensions that declare DependsUponMavenPlugin

to avoid unecessary access to the filesystem by JacocoMavenInitializer

1 files changed, 6 insertions, 1 deletions

diff --git a/sonar-deprecated/src/main/java/org/sonar/api/batch/BatchExtensionDictionnary.java b/sonar-deprecated/src/main/java/org/sonar/api/batch/BatchExtensionDictionnary.java
index bc88a95116d..1c6a3c73158 100644
--- a/sonar-deprecated/src/main/java/org/sonar/api/batch/BatchExtensionDictionnary.java
+++ b/sonar-deprecated/src/main/java/org/sonar/api/batch/BatchExtensionDictionnary.java
@@ -65,7 +65,12 @@ public class BatchExtensionDictionnary {
   }
 
   public Collection<MavenPluginHandler> selectMavenPluginHandlers(Project project) {
-    Collection<DependsUponMavenPlugin> selectedExtensions = select(DependsUponMavenPlugin.class, project, true);
+    List<DependsUponMavenPlugin> selectedExtensions = Lists.newArrayList();
+    for (BatchExtension extension : getExtensions()) {
+      if (ClassUtils.isAssignable(extension.getClass(), DependsUponMavenPlugin.class)) {
+        selectedExtensions.add((DependsUponMavenPlugin) extension);
+      }
+    }
     List<MavenPluginHandler> handlers = Lists.newArrayList();
     for (DependsUponMavenPlugin extension : selectedExtensions) {
       MavenPluginHandler handler = extension.getMavenPluginHandler(project);