Ticket #4720: syndication-500-2-alt.diff

django/contrib/syndication/feeds.py

@@ -55 +55 @@
                 return attr()
         return attr
 
-    def get_feed(self, url=None):
+    def get_object(self, bits):
+        return None
+
+    def get_feed(self, url=""):
         """
         Returns a feedgenerator.DefaultFeed object, fully populated, for
         this feed. Raises FeedDoesNotExist for invalid parameters.
         """
-        if url:
-            try:
-                obj = self.get_object(url.split('/'))
-            except (AttributeError, ObjectDoesNotExist):
-                raise FeedDoesNotExist
-        else:
-            obj = None
+        try:
+            obj = self.get_object([bit for bit in url.split('/') if bit])
+        except (AttributeError, ObjectDoesNotExist):
+            raise FeedDoesNotExist
 
         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.test.client import Client
+
+class SyndicationFeedTest(TestCase):
+    def test_complex_base_url(self):
+        """
+        tests that that the base url for a complex feed doesn't raise a 500
+        exception
+        """
+        c = Client()
+        response = c.get('/syndication/feeds/complex/')
+        self.assertEquals(response.status_code, 404)

tests/regressiontests/syndication/urls.py

@@ +1 @@
+from django.conf.urls.defaults import patterns
+from django.core.exceptions import ObjectDoesNotExist
+from django.contrib.syndication import feeds
+
+
+class ComplexFeed(feeds.Feed):
+    def get_object(self, bits):
+        if len(bits) != 1:
+            raise ObjectDoesNotExist
+        return None
+
+
+urlpatterns = patterns('',
+    (r'^feeds/(?P<url>.*)/$', 'django.contrib.syndication.views.feed', {
+        'feed_dict': dict(
+            complex = ComplexFeed,
+        )}),
+)

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

tests/urls.py

@@ -14 +14 @@
     
     # django built-in views
     (r'^views/', include('regressiontests.views.urls')),
+
+    # test urlconf for syndication tests
+    (r'^syndication/', include('regressiontests.syndication.urls')),
 )