From 6f6dce4bef25846d6efcbf75e1a3aa50bbe77d77 Mon Sep 17 00:00:00 2001 From: Stephen Connolly Date: Thu, 10 Apr 2014 14:28:37 +0100 Subject: [PATCH] [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 --- war/src/main/webapp/scripts/hudson-behavior.js | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/war/src/main/webapp/scripts/hudson-behavior.js b/war/src/main/webapp/scripts/hudson-behavior.js index d7edeb34fd..06f0674475 100644 --- 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); -- GitLab