[FIXED JENKINS-22397] Allow a Trigger to be a DependencyDeclarer.

jglick · jglick · commit efd2b990fbb1 · 2014-03-27T21:43:30.000-04:00
diff --git a/changelog.html b/changelog.html
@@ -61,6 +61,9 @@
   <li class=bug>
     <code>NoSuchMethodError: hudson.model.BuildAuthorizationToken.checkPermission(…)</code> from Build Token Root plugin since 1.556.
     (<a href="https://issues.jenkins-ci.org/browse/JENKINS-22382">issue 22382</a>)
+  <li class=rfe>
+    Allow a <code>Trigger</code> to be a <code>DependencyDeclarer</code>.
+    (<a href="https://issues.jenkins-ci.org/browse/JENKINS-22397">issue 22397</a>)
   <li class=bug>
     Fixed a slow down in resource loading caused by fix to JENKINS-18677.
     (<a href="https://issues.jenkins-ci.org/browse/JENKINS-21579">issue 21579</a>)
diff --git a/core/src/main/java/hudson/matrix/MatrixProject.java b/core/src/main/java/hudson/matrix/MatrixProject.java
@@ -802,7 +802,8 @@ public boolean isFingerprintConfigured() {
         return false;
     }
 
-    protected void buildDependencyGraph(DependencyGraph graph) {
+    @Override protected void buildDependencyGraph(DependencyGraph graph) {
+        super.buildDependencyGraph(graph);
         publishers.buildDependencyGraph(this,graph);
         builders.buildDependencyGraph(this,graph);
         buildWrappers.buildDependencyGraph(this,graph);
diff --git a/core/src/main/java/hudson/model/AbstractProject.java b/core/src/main/java/hudson/model/AbstractProject.java
@@ -1712,9 +1712,11 @@ private void checkAndRecord(AbstractProject that, TreeMap<Integer, RangeSet> r,
 
     /**
      * Builds the dependency graph.
-     * @see DependencyGraph
+     * Since 1.558, not abstract and by default includes dependencies contributed by {@link #triggers()}.
      */
-    protected abstract void buildDependencyGraph(DependencyGraph graph);
+    protected void buildDependencyGraph(DependencyGraph graph) {
+        triggers().buildDependencyGraph(this, graph);
+    }
 
     @Override
     protected SearchIndexBuilder makeSearchIndex() {
diff --git a/core/src/main/java/hudson/model/Project.java b/core/src/main/java/hudson/model/Project.java
@@ -175,7 +175,8 @@ public Publisher getPublisher(Descriptor<Publisher> descriptor) {
         return null;
     }
 
-    protected void buildDependencyGraph(DependencyGraph graph) {
+    @Override protected void buildDependencyGraph(DependencyGraph graph) {
+        super.buildDependencyGraph(graph);
         getPublishersList().buildDependencyGraph(this,graph);
         getBuildersList().buildDependencyGraph(this,graph);
         getBuildWrappersList().buildDependencyGraph(this,graph);
diff --git a/core/src/main/java/jenkins/model/DependencyDeclarer.java b/core/src/main/java/jenkins/model/DependencyDeclarer.java
@@ -25,18 +25,20 @@
 
 import hudson.model.AbstractProject;
 import hudson.model.DependencyGraph;
-import hudson.tasks.BuildStep;
+import hudson.tasks.BuildWrapper;
 import hudson.tasks.Builder;
 import hudson.tasks.Publisher;
+import hudson.triggers.Trigger;
+import hudson.util.DescribableList;
 
 /**
- * Marker interface for those {@link BuildStep}s that can participate
+ * Marker interface for project-associated objects that can participate
  * in the dependency graph computation process.
  *
  * <p>
- * {@link Publisher}s, {@link Builder}s, and {@link hudson.model.JobProperty}s
- * can additional implement this method to add additional edges
- * to the dependency graph computation.
+ * {@link Publisher}s, {@link Builder}s, {@link BuildWrapper}s, and {@link Trigger}s
+ * (or whatever {@link DescribableList#buildDependencyGraph} is called on, by {@link AbstractProject#buildDependencyGraph})
+ * can implement this interface to add additional edges to the dependency graph.
  *
  * @author Nicolas Lalevee
  * @author Martin Ficker

Original file line number	Diff line number	Diff line change
`@@ -802,7 +802,8 @@ public boolean isFingerprintConfigured() {`
`802`	`802`	`return false;`
`803`	`803`	`}`
`804`	`804`
`805`		`- protected void buildDependencyGraph(DependencyGraph graph) {`
	`805`	`+ @Override protected void buildDependencyGraph(DependencyGraph graph) {`
	`806`	`+ super.buildDependencyGraph(graph);`
`806`	`807`	`publishers.buildDependencyGraph(this,graph);`
`807`	`808`	`builders.buildDependencyGraph(this,graph);`
`808`	`809`	`buildWrappers.buildDependencyGraph(this,graph);`
Original file line number	Diff line number	Diff line change
`@@ -175,7 +175,8 @@ public Publisher getPublisher(Descriptor<Publisher> descriptor) {`
`175`	`175`	`return null;`
`176`	`176`	`}`
`177`	`177`
`178`		`- protected void buildDependencyGraph(DependencyGraph graph) {`
	`178`	`+ @Override protected void buildDependencyGraph(DependencyGraph graph) {`
	`179`	`+ super.buildDependencyGraph(graph);`
`179`	`180`	`getPublishersList().buildDependencyGraph(this,graph);`
`180`	`181`	`getBuildersList().buildDependencyGraph(this,graph);`
`181`	`182`	`getBuildWrappersList().buildDependencyGraph(this,graph);`