Favor the new style of form submission over the legacy form to fix: J…

…ENKINS-25533
jenkinsci · Dec 2, 2014 · 15a24dd · 15a24dd
1 parent cee1501
commit 15a24dd
Showing 1 changed file with 5 additions and 4 deletions.
diff --git a/core/src/main/java/hudson/model/AbstractProject.java b/core/src/main/java/hudson/model/AbstractProject.java
@@ -1836,13 +1836,14 @@ protected void submit(StaplerRequest req, StaplerResponse rsp) throws IOExceptio
         else
             scmCheckoutStrategy = null;
 
-        if(req.hasParameter("_.assignedLabelString")) {
+        if(json.optBoolean("hasSlaveAffinity", json.has("label"))) {
+            assignedNode = Util.fixEmptyAndTrim(json.optString("label"));
+        } else if(req.hasParameter("_.assignedLabelString")) {
             // Workaround for JENKINS-25372 while plugin is being updated.
+            // Keep this condition second for JENKINS-25533
             LOGGER.log(Level.WARNING, "label assignment is using legacy '_.assignedLabelString'");
             assignedNode = Util.fixEmptyAndTrim(req.getParameter("_.assignedLabelString"));
-        } else if(json.optBoolean("hasSlaveAffinity", json.has("label"))) {
-            assignedNode = Util.fixEmptyAndTrim(json.optString("label"));
-        } else {
+        } else  {
             assignedNode = null;
         }
         canRoam = assignedNode==null;