[FIXED JENKINS-17110] Build Now should be GET for parameterized jobs.

jenkinsci · Apr 9, 2013 · 97da161 · 97da161
1 parent 7251965
commit 97da161
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 2 deletions.
diff --git a/changelog.html b/changelog.html
@@ -55,7 +55,9 @@
 <!-- Record your changes in the trunk here. -->
 <div id="trunk" style="display:none"><!--=TRUNK-BEGIN=-->
 <ul class=image>
-  <li class=>
+  <li class=bug>
+    “Build Now” context menu item broken for parameterized jobs.
+    (<a href="https://issues.jenkins-ci.org/browse/JENKINS-17110">issue 17110</a>)
 </ul>
 </div><!--=TRUNK-END=-->
 

diff --git a/core/src/main/resources/hudson/model/AbstractProject/sidepanel.jelly b/core/src/main/resources/hudson/model/AbstractProject/sidepanel.jelly
@@ -46,7 +46,7 @@ THE SOFTWARE.
       <j:if test="${it.configurable}">
         <j:if test="${it.buildable}">
           <l:task icon="images/24x24/clock.png" href="${url}/build?delay=0sec" title="${it.buildNowText}"
-                  onclick="${it.parameterized?null:'return build(this)'}" post="true" permission="${it.BUILD}"/>
+                  onclick="${it.parameterized?null:'return build(this)'}" post="${!it.parameterized}" permission="${it.BUILD}"/>
           <script>
             function build(a) {
               new Ajax.Request(a.href);