From cdef13254a2bd5c104ddbb392545828879865c16 Mon Sep 17 00:00:00 2001
From: Igor Pashev <pashev.igor@gmail.com>
Date: Sat, 8 Sep 2018 13:01:29 +0300
Subject: Really allow rebuilding pull requests

(With the Rebuild plugin - https://wiki.jenkins.io/display/JENKINS/Rebuild+Plugin)

* Make builds implicitly parameterized.

* Make sure `bbprb*` are passed in environment and preserved.

* Update Jenkins requirements to 2.60+
---
 .../plugins/bbprb/BitbucketBuildTrigger.java       | 58 +++++++++++++++-------
 1 file changed, 39 insertions(+), 19 deletions(-)

(limited to 'src/main/java/org/jenkinsci/plugins/bbprb/BitbucketBuildTrigger.java')

diff --git a/src/main/java/org/jenkinsci/plugins/bbprb/BitbucketBuildTrigger.java b/src/main/java/org/jenkinsci/plugins/bbprb/BitbucketBuildTrigger.java
index cc4b285..5481aee 100644
--- a/src/main/java/org/jenkinsci/plugins/bbprb/BitbucketBuildTrigger.java
+++ b/src/main/java/org/jenkinsci/plugins/bbprb/BitbucketBuildTrigger.java
@@ -20,7 +20,7 @@ import hudson.model.queue.QueueTaskFuture;
 import hudson.model.Queue;
 import hudson.model.Result;
 import hudson.model.Run;
-import hudson.plugins.git.RevisionParameterAction;
+import hudson.model.StringParameterValue;
 import hudson.security.ACL;
 import hudson.triggers.Trigger;
 import hudson.triggers.TriggerDescriptor;
@@ -35,7 +35,6 @@ import java.util.Map;
 import javax.annotation.Nonnull;
 import javax.annotation.Nullable;
 import jenkins.model.Jenkins;
-import jenkins.model.ParameterizedJobMixIn;
 import net.sf.json.JSONObject;
 import org.acegisecurity.context.SecurityContext;
 import org.acegisecurity.context.SecurityContextHolder;
@@ -75,6 +74,21 @@ public class BitbucketBuildTrigger extends Trigger<AbstractProject<?, ?>> {
 
   private transient ApiClient apiClient;
 
+  private static final transient ArrayList<String> bbprbSafeParameters =
+      new ArrayList<String>() {
+        {
+          add("bbprbDestinationBranch");
+          add("bbprbDestinationCommitHash");
+          add("bbprbDestinationRepository");
+          add("bbprbPullRequestAuthor");
+          add("bbprbPullRequestId");
+          add("bbprbPullRequestTitle");
+          add("bbprbSourceBranch");
+          add("bbprbSourceCommitHash");
+          add("bbprbSourceRepository");
+        }
+      };
+
   public static final BitbucketBuildTriggerDescriptor descriptor =
       new BitbucketBuildTriggerDescriptor();
 
@@ -143,11 +157,31 @@ public class BitbucketBuildTrigger extends Trigger<AbstractProject<?, ?>> {
       SecurityContextHolder.setContext(orig);
     }
 
+    List<ParameterValue> bbprb = new ArrayList<>();
+
+    bbprb.add(new StringParameterValue("bbprbDestinationBranch",
+                                       cause.getDestinationBranch()));
+    bbprb.add(new StringParameterValue("bbprbDestinationCommitHash",
+                                       cause.getDestinationCommitHash()));
+    bbprb.add(new StringParameterValue("bbprbDestinationRepository",
+                                       cause.getDestinationRepository()));
+    bbprb.add(new StringParameterValue("bbprbPullRequestAuthor",
+                                       cause.getPullRequestAuthor()));
+    bbprb.add(new StringParameterValue("bbprbPullRequestId",
+                                       cause.getPullRequestId()));
+    bbprb.add(new StringParameterValue("bbprbPullRequestTitle",
+                                       cause.getPullRequestTitle()));
+    bbprb.add(
+        new StringParameterValue("bbprbSourceBranch", cause.getSourceBranch()));
+    bbprb.add(new StringParameterValue("bbprbSourceCommitHash",
+                                       cause.getSourceCommitHash()));
+    bbprb.add(new StringParameterValue("bbprbSourceRepository",
+                                       cause.getSourceRepository()));
+
     setPRState(cause, BuildState.INPROGRESS, this.job.getUrl());
 
-    this.job.scheduleBuild2(
-        0, cause, new ParametersAction(this.getDefaultParameters()),
-        new RevisionParameterAction(cause.getSourceCommitHash()));
+    this.job.scheduleBuild2(0, cause,
+                            new ParametersAction(bbprb, bbprbSafeParameters));
   }
 
   private void
@@ -222,20 +256,6 @@ public class BitbucketBuildTrigger extends Trigger<AbstractProject<?, ?>> {
     return false;
   }
 
-  private ArrayList<ParameterValue> getDefaultParameters() {
-    Map<String, ParameterValue> values = new HashMap<String, ParameterValue>();
-    ParametersDefinitionProperty definitionProperty =
-        this.job.getProperty(ParametersDefinitionProperty.class);
-
-    if (definitionProperty != null) {
-      for (ParameterDefinition definition :
-           definitionProperty.getParameterDefinitions()) {
-        values.put(definition.getName(), definition.getDefaultParameterValue());
-      }
-    }
-    return new ArrayList<ParameterValue>(values.values());
-  }
-
   public void handlePR(JSONObject pr) {
     JSONObject src = pr.getJSONObject("source");
     JSONObject dst = pr.getJSONObject("destination");
-- 
cgit v1.2.3