From 43ac50d92d3d21f029ea8b10a6d2983ca7aecc72 Mon Sep 17 00:00:00 2001 From: Kirill Spitsin Date: Fri, 24 May 2013 16:51:07 +0200 Subject: [PATCH] Changed View.add_job() to actually add job to the view, rather than replace all jobs in the view by that job. --- jenkinsapi/view.py | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/jenkinsapi/view.py b/jenkinsapi/view.py index e489006..b270f31 100644 --- a/jenkinsapi/view.py +++ b/jenkinsapi/view.py @@ -62,6 +62,10 @@ class View(JenkinsBase): elif not self.get_jenkins_obj().has_job(str_job_name): return "Job %s is not known - available: %s" % ( str_job_name, ", ".join(self.get_jenkins_obj().get_jobs_list())) else: + def get_job_url(name): + return self.jenkins_obj.get_job(name).baseurl + jobs = self._data.setdefault('jobs', []) + jobs.append({'name': str_job_name, 'url': get_job_url(str_job_name)}) data = { "description":"", "statusFilter":"", @@ -79,7 +83,6 @@ class View(JenkinsBase): data["name"] = self.name for job in self.get_job_dict().keys(): data[job]='on' - data[str_job_name] = "on" data['json'] = data.copy() self.post_data('%sconfigSubmit' % self.baseurl, urllib.urlencode(data)) return "Job %s is add in View %s successful" % (str_job_name, self.baseurl) -- 2.7.4