[JENKINS-21268] Don't reference maven classes if plugin is deactivated.

jenkinsci · Jan 11, 2014 · c650bee · c650bee
1 parent 686ea33
commit c650bee
Show file tree

Hide file tree

Showing 3 changed files with 34 additions and 3 deletions.
diff --git a/pom.xml b/pom.xml
@@ -4,7 +4,7 @@
   <parent>
     <groupId>org.jvnet.hudson.plugins</groupId>
     <artifactId>analysis-pom</artifactId>
-    <version>1.47</version>
+    <version>1.48</version>
     <relativePath>../analysis-pom/pom.xml</relativePath>
   </parent>
 
@@ -36,7 +36,7 @@
     <dependency>
       <groupId>org.jvnet.hudson.plugins</groupId>
       <artifactId>analysis-core</artifactId>
-      <version>1.54</version>
+      <version>1.55-SNAPSHOT</version>
     </dependency>
     <dependency>
       <groupId>org.jvnet.hudson.plugins</groupId>

diff --git a/src/main/java/hudson/plugins/tasks/MavenInitialization.java b/src/main/java/hudson/plugins/tasks/MavenInitialization.java
@@ -0,0 +1,27 @@
+package hudson.plugins.tasks;
+
+import hudson.plugins.analysis.views.DetailFactory;
+
+/**
+ * Initialization of Maven classes.
+ *
+ * @author Ulli Hafner
+ */
+public final class MavenInitialization {
+    /**
+     * Initializes the detail builder for Maven builds.
+     *
+     * @param detailBuilder
+     *            the builder to use
+     */
+    public static void run(final DetailFactory detailBuilder) {
+        DetailFactory.addDetailBuilder(TasksMavenResultAction.class, detailBuilder);
+    }
+
+    /**
+     * Creates a new instance of {@link MavenInitialization}.
+     */
+    private MavenInitialization() {
+        // prevents instantiation
+    }
+}
diff --git a/src/main/java/hudson/plugins/tasks/TasksPlugin.java b/src/main/java/hudson/plugins/tasks/TasksPlugin.java
@@ -1,6 +1,8 @@
 package hudson.plugins.tasks;
 
 import hudson.Plugin;
+
+import hudson.plugins.analysis.core.PluginDescriptor;
 import hudson.plugins.analysis.views.DetailFactory;
 
 /**
@@ -13,6 +15,8 @@ public class TasksPlugin extends Plugin {
     public void start() {
         TasksDetailBuilder detailBuilder = new TasksDetailBuilder();
         DetailFactory.addDetailBuilder(TasksResultAction.class, detailBuilder);
-        DetailFactory.addDetailBuilder(TasksMavenResultAction.class, detailBuilder);
+        if (PluginDescriptor.isMavenPluginInstalled()) {
+            MavenInitialization.run(detailBuilder);
+        }
     }
 }