Skip to content

Commit

Permalink
[FIXED JENKINS-22570] On a configure screen that has multiple groups …
Browse files Browse the repository at this point in the history
…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

(cherry picked from commit 6f6dce4)
  • Loading branch information
stephenc authored and olivergondza committed May 5, 2014
1 parent 1da70d3 commit 1335246
Showing 1 changed file with 4 additions and 3 deletions.
7 changes: 4 additions & 3 deletions war/src/main/webapp/scripts/hudson-behavior.js
Expand Up @@ -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);
Expand Down

0 comments on commit 1335246

Please sign in to comment.