[Checkins] SVN: lovely.remotetask/trunk/ - Using the correct plural form of status (which is status) in ITaskService.clean

Christian Zagrodnick cz at gocept.com
Mon Mar 2 04:02:04 EST 2009


Log message for revision 97411:
  - Using the correct plural form of status (which is status) in  ITaskService.clean
  
  

Changed:
  U   lovely.remotetask/trunk/CHANGES.txt
  U   lovely.remotetask/trunk/src/lovely/remotetask/service.py

-=-
Modified: lovely.remotetask/trunk/CHANGES.txt
===================================================================
--- lovely.remotetask/trunk/CHANGES.txt	2009-03-02 09:01:07 UTC (rev 97410)
+++ lovely.remotetask/trunk/CHANGES.txt	2009-03-02 09:02:04 UTC (rev 97411)
@@ -17,7 +17,10 @@
 
 - Changed URL to pypi.
 
+- Using the correct plural form of status (which is status) in
+  ITaskService.clean
 
+
 2008/11/07 0.2.15a1:
 ====================
 

Modified: lovely.remotetask/trunk/src/lovely/remotetask/service.py
===================================================================
--- lovely.remotetask/trunk/src/lovely/remotetask/service.py	2009-03-02 09:01:07 UTC (rev 97410)
+++ lovely.remotetask/trunk/src/lovely/remotetask/service.py	2009-03-02 09:02:04 UTC (rev 97411)
@@ -110,14 +110,14 @@
     def reschedule(self, jobid):
         self._scheduledQueue.put(self.jobs[jobid])
 
-    def clean(self, stati=[interfaces.CANCELLED, interfaces.ERROR,
-                           interfaces.COMPLETED]):
+    def clean(self, status=[interfaces.CANCELLED, interfaces.ERROR,
+                            interfaces.COMPLETED]):
         """See interfaces.ITaskService"""
         allowed = [interfaces.CANCELLED, interfaces.ERROR,
                    interfaces.COMPLETED]
         for key in list(self.jobs.keys()):
             job = self.jobs[key]
-            if job.status in stati:
+            if job.status in status:
                 if job.status not in allowed:
                     raise ValueError('Not allowed status for removing. %s' % \
                         job.status)



More information about the Checkins mailing list