SONAR-926 No more call shouldExecuteOnProject on extensions that declare DependsUponM...

to avoid unecessary access to the filesystem by JacocoMavenInitializer

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 bc88a95116d451914eb4d6217e3acc1a877ec8b8..1c6a3c73158bf06870398843b8bf10fd3bbbd5d6 100644 (file)
--- 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);