Changeset 12583

Timestamp:

10 Feb 2015, 14:33:23 (10 years ago)

Author:

uli

Message:

Do not store empty jobs.

File:

• main/waeup.kofa/trunk/src/waeup/kofa/reports.py (modified) (2 diffs)

main/waeup.kofa/trunk/src/waeup/kofa/reports.py

@@ -268 +268 @@
         persistent list.
 
-        Returns the job ID of the job started.
+        Returns the job ID of the job started, `None` if the job could
+        not be started.
         """
         site = grok.getSite()
@@ -274 +275 @@
         job = AsyncReportJob(site, generator_name, args=args, kw=kw)
         job_id = manager.put(job)
-        # Make sure that the persisted list is stored in ZODB
-        self.running_report_jobs = PersistentList(self.running_report_jobs)
-        self.running_report_jobs.append((job_id, generator_name, user_id),)
+        if job_id is not None:
+            # Make sure that the persisted list is stored in ZODB
+            self.running_report_jobs = PersistentList(self.running_report_jobs)
+            self.running_report_jobs.append((job_id, generator_name, user_id),)
         return job_id