[FIXED JENKINS-22570] On a configure screen that has multiple groups …

…of radio buttons, clicking the apply button clears all but the last radio group selection - Always run git status before and after you commit, just in case there are changes you made since your last call to git add - At least the integration test cases caught this one though
jenkinsci · Apr 10, 2014 · 6f6dce4 · 6f6dce4
1 parent 88d84f6
commit 6f6dce4
Showing 1 changed file with 4 additions and 3 deletions.
diff --git a/war/src/main/webapp/scripts/hudson-behavior.js b/war/src/main/webapp/scripts/hudson-behavior.js
@@ -1865,13 +1865,14 @@ function buildFormTree(form) {
                 r=0;
                 while (e.name.substring(r,r+8)=='removeme')
                     r = e.name.indexOf('_',r+8)+1;
+                p = findParent(e);
                 if(e.groupingNode) {
-                    p = findParent(e);
                     addProperty(p, e.name.substring(r), e.formDom = { value: e.value });
-                    break;
+                } else {
+                    addProperty(p, e.name.substring(r), e.value);
                 }
+                break;
 
-                // otherwise fall through
             default:
                 p = findParent(e);
                 addProperty(p, e.name, e.value);