JENKINS-38084 - Add support to mark jobs as disabled on import

jenkinsci · Sep 12, 2016 · 144aa1d · 144aa1d
1 parent 2b4a0f9
commit 144aa1d
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 1 deletion.
diff --git a/src/main/java/org/jenkins/ci/plugins/jobimport/JobImportAction.java b/src/main/java/org/jenkins/ci/plugins/jobimport/JobImportAction.java
@@ -36,6 +36,7 @@
 import hudson.model.Item;
 import hudson.model.RootAction;
 import hudson.model.TopLevelItem;
+import hudson.model.AbstractProject;
 import hudson.security.ACL;
 import hudson.util.FormValidation;
 import java.io.IOException;
@@ -125,7 +126,12 @@ public void doImport(final StaplerRequest request, final StaplerResponse respons
 
               try {
                   inputStream = URLUtils.fetchUrl(remoteJob.getUrl() + "/config.xml", username, password);
-                  Jenkins.getInstance().createProjectFromXML(remoteJob.getName(), inputStream);
+                  TopLevelItem topLevelItem = Jenkins.getInstance().createProjectFromXML(remoteJob.getName(), inputStream);
+                  boolean disableProject = request.getParameter("disable-" + jobUrl) != null;
+                  if (topLevelItem instanceof AbstractProject && disableProject) {
+                    AbstractProject project = (AbstractProject)topLevelItem;
+                    project.disable();
+                  }
                   remoteJobsImportStatus.get(remoteJob).setStatus(MessagesUtils.formatSuccess());
               }
 

diff --git a/src/main/resources/org/jenkins/ci/plugins/jobimport/JobImportAction/index.jelly b/src/main/resources/org/jenkins/ci/plugins/jobimport/JobImportAction/index.jelly
@@ -60,12 +60,14 @@
               <table class="jobImportPlugin" cellpadding="3" cellspacing="3">
                 <tr>
                   <th>Import?</th>
+                  <th>Disable?</th>
                   <th>Name</th>
                   <th>Description</th>
                 </tr>
                 <j:forEach var="job" items="${it.remoteJobs}">
                   <tr>
                     <td><f:checkbox name="jobUrl" value="${job.url}"/></td>
+                    <td><f:checkbox name="disable-${job.url}" /></td>
                     <td><a target="_blank" href="${job.url}">${job.name}</a></td>
                     <td>${job.description}</td>
                   </tr>