bug fix '' instead of none for nonexistent param

open-contracting · Mar 18, 2021 · 2c3a471 · 2c3a471
1 parent 98f339d
commit 2c3a471
Showing 1 changed file with 4 additions and 4 deletions.
diff --git a/kingfisher_scrapy/extensions.py b/kingfisher_scrapy/extensions.py
@@ -360,7 +360,7 @@ def spider_closed(self, spider, reason):
         def response_callback(response):
             if not response.code == 200:
                 spider.logger.warning(
-                    'Failed to post close collection. API status code: {}'.format(response.code))
+                    "Failed to post close collection. API status code: {}".format(response.code))
 
         self._post_async("api/v1/close_collection", data, response_callback)
 
@@ -373,13 +373,13 @@ def item_scraped(self, item, spider):
 
         data = {
             "collection_id": self.collection_id,
-            "path": os.path.join(item.get("files_store", None), item.get("path", None)),
-            "url": item.get('url', None)
+            "path": os.path.join(item.get("files_store", ""), item.get("path", "")),
+            "url": item.get("url", None)
         }
 
         if isinstance(item, FileError):
             # in case of error send info about it to api
-            data['errors'] = json.dumps(item.get("errors", None))
+            data["errors"] = json.dumps(item.get("errors", None))
 
         def response_callback(response):
             if not response.code == 200: