From bdd627fe1ddce6610e593cb66e16a611e6e0392e Mon Sep 17 00:00:00 2001 From: Daniel Grana Date: Fri, 15 Jul 2011 15:17:38 -0300 Subject: [PATCH] allow overriding store_uri by extending ImagePipeline --HG-- extra : rebase_source : 5c561b8282f733ab0f26607059dd96d858154426 --- scrapy/contrib/pipeline/images.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/scrapy/contrib/pipeline/images.py b/scrapy/contrib/pipeline/images.py index 427222d47..c7333a542 100644 --- a/scrapy/contrib/pipeline/images.py +++ b/scrapy/contrib/pipeline/images.py @@ -154,6 +154,8 @@ class ImagesPipeline(MediaPipeline): } def __init__(self, store_uri, download_func=None): + if not store_uri: + raise NotConfigured self.store = self._get_store(store_uri) super(ImagesPipeline, self).__init__(download_func=download_func) @@ -167,8 +169,6 @@ class ImagesPipeline(MediaPipeline): s3store.AWS_ACCESS_KEY_ID = settings['AWS_ACCESS_KEY_ID'] s3store.AWS_SECRET_ACCESS_KEY = settings['AWS_SECRET_ACCESS_KEY'] store_uri = settings['IMAGES_STORE'] - if not store_uri: - raise NotConfigured return cls(store_uri) def _get_store(self, uri):