Ticket #4720: syndication-500-2.diff

django/contrib/syndication/feeds.py

@@ -55 +55 @@
-                return attr()
-        return attr
-
-
+
+
+
+
-        """
-        Returns a feedgenerator.DefaultFeed object, fully populated, for
-        this feed. Raises FeedDoesNotExist for invalid parameters.
-        """
-
-
-
-
-
-
-
+
+
+
+
-
-        if Site._meta.installed:
-            current_site = Site.objects.get_current()

tests/regressiontests/syndication/tests.py

@@ +1 @@
+# -*- coding: utf-8 -*-
+
+from django.test import TestCase 
+from django.contrib.syndication import feeds
+from django.core.exceptions import ObjectDoesNotExist
+from django.http import HttpRequest
+
+class SyndicationFeedTest(TestCase):
+    def test_send_empty_urls_to_get_object(self):
+        """
+        tests that get_object gets a chance to handle empty urls
+        """
+        _test_log = []
+        class FeedStub(feeds.Feed):
+            def get_object(self, bits):
+                _test_log.append(bits)
+                raise ObjectDoesNotExist
+                
+        f = FeedStub('foo', HttpRequest())
+        self.assertRaises(
+            feeds.FeedDoesNotExist,
+            f.get_feed,
+            '')
+        self.assertEquals(_test_log, [[]])

docs/syndication_feeds.txt

@@ -243 +243 @@
-      raises ``Beat.DoesNotExist`` on failure, and ``Beat.DoesNotExist`` is a
-      subclass of ``ObjectDoesNotExist``. Raising ``ObjectDoesNotExist`` in
-      ``get_object()`` tells Django to produce a 404 error for that request.
+
+      **New in Django development version:** Note that get_object also gets a
+      chance to handle the ``/rss/beats/`` url.  In this case bits will be
+      an empty list.  In our example, since ``len(bits) != 1`` an
+      ``ObjectDoesNotExist`` exception will be raised, so ``/rss/beats/`` will
+      generate a 404 page.  But you can handle this case however you like.  For
+      example you could generate a combined feed for all beats.
+
-    * To generate the feed's ``<title>``, ``<link>`` and ``<description>``,
-      Django uses the ``title()``, ``link()`` and ``description()`` methods. In
-      the previous example, they were simple string class attributes, but this