From 2b93b0a93c914ab4994989d74eb33d45f22ea50e Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Libor=20Nenad=E1l?= Date: Sat, 5 May 2012 14:53:02 +0200 Subject: [PATCH] Do not try to set item['images'] if it is not defined in the Item. --- scrapy/contrib/pipeline/images.py | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/scrapy/contrib/pipeline/images.py b/scrapy/contrib/pipeline/images.py index e1d183e0f..29a5910e7 100644 --- a/scrapy/contrib/pipeline/images.py +++ b/scrapy/contrib/pipeline/images.py @@ -309,5 +309,6 @@ class ImagesPipeline(MediaPipeline): return [Request(x) for x in item.get('image_urls', [])] def item_completed(self, results, item, info): - item['images'] = [x for ok, x in results if ok] + if 'images' in item.fields: + item['images'] = [x for ok, x in results if ok] return item