Mv current update_tags logic to Jobs

Author: vshlapakov

README.rst

@@ -460,11 +460,11 @@ To mark a job with tag ``consumed``::
 
 To mark all spider jobs with tag ``consumed``::
 
-    >>> spider.update_tags(add=['consumed'])
+    >>> spider.jobs.update_tags(add=['consumed'])
 
 To remove existing tag ``existing`` for all spider jobs::
 
-    >>> spider.update_tags(remove=['existing'])
+    >>> spider.jobs.update_tags(remove=['existing'])
 
 Modifying tags is available on spider/job levels.
 

scrapinghub/client.py

@@ -129,14 +129,6 @@ def __init__(self, client, projectid, spiderid, spidername):
         self.jobs = Jobs(client, projectid, self)
         self._client = client
 
-    def update_tags(self, add=None, remove=None):
-        params = get_tags_for_update(add_tag=add, remove_tag=remove)
-        if not params:
-            return
-        params.update({'project': self.projectid, 'spider': self.name})
-        result = self._client._connection._post('jobs_update', 'json', params)
-        return result['count']
-
 
 class Jobs(object):
 
@@ -203,6 +195,17 @@ def _extract_spider_id(self, params):
             raise ValueError('Please use same spider id')
         return spiderid
 
+    def update_tags(self, add=None, remove=None, spidername=None):
+        spidername = spidername or (self.spider.name if self.spider else None)
+        if not spidername:
+            raise ValueError('Please provide spidername')
+        params = get_tags_for_update(add_tag=add, remove_tag=remove)
+        if not params:
+            return
+        params.update({'project': self.projectid, 'spider': spidername})
+        result = self._client._connection._post('jobs_update', 'json', params)
+        return result['count']
+
 
 class Job(object):