Only send selected change data back to the server

Use a separate form for the submit, containing only the hidden field that is used by the serverside code. Fixes: JENKINS-21064
jenkinsci · Sep 16, 2014 · 4ce1427 · 4ce1427
1 parent e86cd84
commit 4ce1427
Showing 1 changed file with 4 additions and 2 deletions.
diff --git a/...udson/plugins/gerrit/trigger/hudsontrigger/actions/manual/ManualTriggerAction/index.jelly b/...udson/plugins/gerrit/trigger/hudsontrigger/actions/manual/ManualTriggerAction/index.jelly
@@ -80,13 +80,12 @@
                                 </p>
                             </f:block>
                         </j:if>
-                        <f:form method="post" action="build" name="theBuild">
+                        <f:form method="" action="" name="">
                             <f:section title="${%Search Result}">
                                 <f:description>
                                     ${%SearchResultDescription}
                                 </f:description>
                                 <f:block>
-                                    <input type="hidden" id="selectedIds" name="selectedIds" value=""/>
 
                                     <j:set var="result" value="${request.session.getAttribute('result')}"/>
                                     <table width="100%" border="1" cellpadding="2" cellspacing="0"
@@ -260,6 +259,9 @@
                                     -->
                                 </f:block>
                             </f:section>
+                        </f:form>
+                        <f:form method="post" action="build" name="theBuild">
+                            <input type="hidden" id="selectedIds" name="selectedIds" value=""/>
                             <f:bottomButtonBar>
                                 <f:submit value="${%Trigger Selected}"/>
                             </f:bottomButtonBar>