Skip to content

Commit

Permalink
Browse files Browse the repository at this point in the history
[JENKINS-38928] Handle radio groups
This fixes the form handling issue after 'Apply' is clicked.
  • Loading branch information
Vlatombe committed Nov 3, 2016
1 parent a451868 commit e1b2326
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 5 deletions.
9 changes: 4 additions & 5 deletions Jenkinsfile
Expand Up @@ -4,7 +4,7 @@ node('hi-speed') {
dir('plugin') {
checkout scm
timeout(30) {
withMaven(jdk: 'Oracle JDK 1.8 (latest)',
withMaven(jdk: 'Oracle JDK 8 (latest)',
maven: 'Maven 3.2.1') {
sh 'mvn clean install -B'
formElementPathVersion = sh script: 'cat target/form-element-path/META-INF/MANIFEST.MF | grep Implementation-Version | cut -f 2 -d ":" | xargs', returnStdout: true
Expand All @@ -19,7 +19,7 @@ node('hi-speed') {
filter: 'war/target/jenkins.war',
fingerprintArtifacts: true,
flatten: true,
projectName: 'jenkins_lts_branch',
projectName: 'core/jenkins_lts_branch',
selector: [
$class: 'StatusBuildSelector',
stable: false
Expand All @@ -33,18 +33,17 @@ node('hi-speed') {
'JENKINS_JAVA_HOME=/opt/jdk/jdk1.7.latest/',
'JAVA_HOME=/opt/jdk/openjdk8.latest/',
'EXERCISEDPLUGINREPORTER=textfile',
"FORM_ELEMENT_PATH_VERSION=$formElementPathVersion"]) {
"FORM_ELEMENT_PATH_VERSION=${formElementPathVersion.trim()}"]) {
sh '''
unzip -p $JENKINS_WAR META-INF/MANIFEST.MF | perl -p -0777 -e 's/\\r?\\n //g' | fgrep Jenkins-Version
cp $SAUCE_SECRET ~/.sauce-ondemand || true
firefox --version
mvn clean test -Dmaven.test.failure.ignore=true -DforkCount=1 -Dtest=FreestyleJobTest -B
mvn clean test -Dmaven.test.failure.ignore=true -DforkCount=1 -B
'''
}
}
}
archiveArtifacts artifacts: 'target/exercised-plugins.properties', excludes: null
}
}
}
Expand Up @@ -124,6 +124,7 @@ Behaviour.addLoadEvent(function(){
case "submit":
break;
case "checkbox":
case "radio":
p = findParent(e);
if(e.groupingNode) {
e.formDom = {};
Expand Down

0 comments on commit e1b2326

Please sign in to comment.