From 661aeb5cda2d9e8665a14c4ef8c655329a6a5531 Mon Sep 17 00:00:00 2001
From: Pablo Hoffman <pablo@pablohoffman.com>
Date: Sat, 8 Aug 2009 05:01:18 -0300
Subject: [PATCH] moved ItemPipelineManager from scrapy.item.pipeline to
 scrapy.contrib.pipeline

---
 scrapy/conf/default_settings.py     |  2 +-
 scrapy/contrib/pipeline/__init__.py | 65 +++++++++++++++++++++++++++++
 scrapy/item/pipeline.py             | 64 ----------------------------
 3 files changed, 66 insertions(+), 65 deletions(-)
 delete mode 100644 scrapy/item/pipeline.py

diff --git a/scrapy/conf/default_settings.py b/scrapy/conf/default_settings.py
index 6402c76f5..1e2270d23 100644
--- a/scrapy/conf/default_settings.py
+++ b/scrapy/conf/default_settings.py
@@ -110,7 +110,7 @@ HTTPCACHE_IGNORE_MISSING = False
 HTTPCACHE_SECTORIZE = True
 HTTPCACHE_EXPIRATION_SECS = 0
 
-ITEM_PROCESSOR = 'scrapy.item.pipeline.ItemPipelineManager'
+ITEM_PROCESSOR = 'scrapy.contrib.pipeline.ItemPipelineManager'
 
 # Item pipelines are typically set in specific commands settings
 ITEM_PIPELINES = []
diff --git a/scrapy/contrib/pipeline/__init__.py b/scrapy/contrib/pipeline/__init__.py
index e69de29bb..6efb1bd9b 100644
--- a/scrapy/contrib/pipeline/__init__.py
+++ b/scrapy/contrib/pipeline/__init__.py
@@ -0,0 +1,65 @@
+"""
+Item pipeline
+
+See documentation in docs/item-pipeline.rst
+"""
+
+from scrapy import log
+from scrapy.core.exceptions import NotConfigured
+from scrapy.item.models import BaseItem
+from scrapy.utils.misc import load_object
+from scrapy.utils.defer import defer_succeed, mustbe_deferred
+from scrapy.conf import settings
+
+class ItemPipelineManager(object):
+
+    def __init__(self):
+        self.loaded = False
+        self.enabled = {}
+        self.disabled = {}
+        self.pipeline = []
+        self.load()
+
+    def load(self):
+        """
+        Load pipelines stages defined in settings module
+        """
+        self.enabled.clear()
+        self.disabled.clear()
+        for pipepath in settings.getlist('ITEM_PIPELINES'):
+            cls = load_object(pipepath)
+            if cls:
+                try:
+                    pipe = cls()
+                    self.pipeline.append(pipe)
+                    self.enabled[cls.__name__] = pipe
+                except NotConfigured, e:
+                    self.disabled[cls.__name__] = pipepath
+                    if e.args:
+                        log.msg(e)
+        log.msg("Enabled item pipelines: %s" % ", ".join(self.enabled.keys()),
+            level=log.DEBUG)
+        self.loaded = True
+
+    def open_domain(self, domain):
+        pass
+
+    def close_domain(self, domain):
+        pass
+
+    def process_item(self, item, spider):
+        if not self.pipeline:
+            return defer_succeed(item)
+
+        def next_stage(item, stages_left):
+            assert isinstance(item, BaseItem), \
+                'Item pipelines must return a BaseItem, got %s' % type(item).__name__
+            if not stages_left:
+                return item
+            current_stage = stages_left.pop(0)
+            d = mustbe_deferred(current_stage.process_item, spider.domain_name, item)
+            d.addCallback(next_stage, stages_left)
+            return d
+
+        deferred = mustbe_deferred(next_stage, item, self.pipeline[:])
+        return deferred
diff --git a/scrapy/item/pipeline.py b/scrapy/item/pipeline.py
deleted file mode 100644
index 43e42f6a8..000000000
--- a/scrapy/item/pipeline.py
+++ /dev/null
@@ -1,64 +0,0 @@
-"""
-Item pipeline manager
-
-See documentation in docs/item-pipeline.rst
-"""
-from scrapy import log
-from scrapy.core.exceptions import NotConfigured
-from scrapy.item.models import BaseItem
-from scrapy.utils.misc import load_object
-from scrapy.utils.defer import defer_succeed, mustbe_deferred
-from scrapy.conf import settings
-
-class ItemPipelineManager(object):
-
-    def __init__(self):
-        self.loaded = False
-        self.enabled = {}
-        self.disabled = {}
-        self.pipeline = []
-        self.load()
-
-    def load(self):
-        """
-        Load pipelines stages defined in settings module
-        """
-        self.enabled.clear()
-        self.disabled.clear()
-        for pipepath in settings.getlist('ITEM_PIPELINES'):
-            cls = load_object(pipepath)
-            if cls:
-                try:
-                    pipe = cls()
-                    self.pipeline.append(pipe)
-                    self.enabled[cls.__name__] = pipe
-                except NotConfigured, e:
-                    self.disabled[cls.__name__] = pipepath
-                    if e.args:
-                        log.msg(e)
-        log.msg("Enabled item pipelines: %s" % ", ".join(self.enabled.keys()),
-            level=log.DEBUG)
-        self.loaded = True
-
-    def open_domain(self, domain):
-        pass
-
-    def close_domain(self, domain):
-        pass
-
-    def process_item(self, item, spider):
-        if not self.pipeline:
-            return defer_succeed(item)
-
-        def next_stage(item, stages_left):
-            assert isinstance(item, BaseItem), \
-                'Item pipelines must return a BaseItem, got %s' % type(item).__name__
-            if not stages_left:
-                return item
-            current_stage = stages_left.pop(0)
-            d = mustbe_deferred(current_stage.process_item, spider.domain_name, item)
-            d.addCallback(next_stage, stages_left)
-            return d
-
-        deferred = mustbe_deferred(next_stage, item, self.pipeline[:])
-        return deferred