Merge pull request #654 from tpeng/patch-2

add message when raise IngoreReques; fix item_scraped document

docs/topics/signals.rst

@@ -63,7 +63,7 @@ item_scraped
 ------------
 
 .. signal:: item_scraped
-.. function:: item_scraped(item, response, spider)
+.. function:: item_scraped(item, spider, response)
 
     Sent when an item has been scraped, after it has passed all the
     :ref:`topics-item-pipeline` stages (without being dropped).
@@ -73,12 +73,12 @@ item_scraped
     :param item: the item scraped
     :type item: :class:`~scrapy.item.Item` object
 
-    :param response: the response from where the item was scraped
-    :type response: :class:`~scrapy.http.Response` object
-
     :param spider: the spider which scraped the item
     :type spider: :class:`~scrapy.spider.Spider` object
 
+    :param response: the response from where the item was scraped
+    :type response: :class:`~scrapy.http.Response` object
+
 item_dropped
 ------------
 

scrapy/contrib/downloadermiddleware/redirect.py

@@ -39,7 +39,7 @@ class BaseRedirectMiddleware(object):
         else:
             log.msg(format="Discarding %(request)s: max redirections reached",
                     level=log.DEBUG, spider=spider, request=request)
-            raise IgnoreRequest
+            raise IgnoreRequest("max redirections reached")
 
     def _redirect_request_using_get(self, request, redirect_url):
         redirected = request.replace(url=redirect_url, method='GET', body='')