Merge pull request entagen#50 from yonatanp/master

clone: clones of disabled jobs now stay disabled
andreineculau · Mar 31, 2014 · 364d170 · 364d170
2 parents 451281d + abefd43
commit 364d170
Showing 1 changed file with 5 additions and 2 deletions.
diff --git a/src/main/groovy/com/entagen/jenkins/JenkinsApi.groovy b/src/main/groovy/com/entagen/jenkins/JenkinsApi.groovy
@@ -59,9 +59,12 @@ class JenkinsApi {
         post('createItem', missingJobConfig, [name: missingJob.jobName, mode: 'copy', from: templateJob.jobName], ContentType.XML)
 
         post('job/' + missingJob.jobName + "/config.xml", missingJobConfig, [:], ContentType.XML)
-        //Forced diable enable to work around Jenkins' automatic disabling of clones jobs
+        //Forced disable enable to work around Jenkins' automatic disabling of clones jobs
+        //But only if the original job was enabled
         post('job/' + missingJob.jobName + '/disable')
-        post('job/' + missingJob.jobName + '/enable')
+        if (!missingJobConfig.contains("<disabled>true</disabled>")) {
+            post('job/' + missingJob.jobName + '/enable')
+        }
     }
 
     void startJob(ConcreteJob job) {