RomanNum3ral
/
scrapy
mirror of https://github.com/scrapy/scrapy.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Merge pull request #129 from saxicek/master 

ImagesPipeline should not fail if Item['images'] is not defined.
This commit is contained in:
Daniel Graña 2012-05-07 10:38:49 -07:00
parent 69078368ab 2b93b0a93c
commit 43732e5042
1 changed files with 2 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
scrapy/contrib/pipeline/images.py
View File

@ -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