[patch] allow generic views to respond with non text/html content types

[ian@…]

I needed to have a page respond with a text/plain content type.
here's the patch

Index: views/generic/list_detail.py
===================================================================
--- views/generic/list_detail.py        (revision 2970)
+++ views/generic/list_detail.py        (working copy)
@@ -6,7 +6,8 @@
 
 def object_list(request, queryset, paginate_by=None, allow_empty=False,
         template_name=None, template_loader=loader,
-        extra_context={}, context_processors=None, template_object_name='object'):
+        extra_context={}, context_processors=None, template_object_name='object',
+        mimetype=None):
     """
     Generic list of objects.
 
@@ -73,12 +74,13 @@
         model = queryset.model
         template_name = "%s/%s_list.html" % (model._meta.app_label, model._meta.object_name.lower())
     t = template_loader.get_template(template_name)
-    return HttpResponse(t.render(c))
+    return HttpResponse(t.render(c), mimetype=mimetype)
 
 def object_detail(request, queryset, object_id=None, slug=None,
         slug_field=None, template_name=None, template_name_field=None,
         template_loader=loader, extra_context={},
-        context_processors=None, template_object_name='object'):
+        context_processors=None, template_object_name='object',
+        mimetype=None):
     """
     Generic list of objects.
 
@@ -113,6 +115,6 @@
             c[key] = value()
         else:
             c[key] = value
-    response = HttpResponse(t.render(c))
+    response = HttpResponse(t.render(c), mimetype=mimetype)
     populate_xheaders(request, response, model, getattr(obj, obj._meta.pk.name))
     return response

[EspenG]

Should not a default mimetype be set in stead of None?

[ian@…]

Hi Espen,
passing 'None' to HttpResponse *is* the default, and lets HttpResponse figure out the correct default.

[EspenG]

Sorry about that then. I taugth that when the default was 'None' I had to pass along a mimetype every time, but as you say that was wrong then.

[Adrian Holovaty]

(In [3022]) Fixed #2000 -- Added 'mimetype' parameter to generic views. Thanks, Ian Holsman