From 940d17409e53658c510c1ae4962312b02e5ccaa4 Mon Sep 17 00:00:00 2001 From: Timo Graham Date: Sun, 22 May 2011 00:08:13 +0000 Subject: [PATCH] Fixed #16021 - Minor documentation fixes for Generic Class Views; thanks Bradley Ayers. git-svn-id: http://code.djangoproject.com/svn/django/trunk@16256 bcc190cf-cafb-0310-a4f2-bffc1f526a37 --- docs/ref/class-based-views.txt | 12 ++++++------ docs/topics/class-based-views.txt | 20 ++++++++++---------- 2 files changed, 16 insertions(+), 16 deletions(-) diff --git a/docs/ref/class-based-views.txt b/docs/ref/class-based-views.txt index 22a204d57a7..c83559fdeb0 100644 --- a/docs/ref/class-based-views.txt +++ b/docs/ref/class-based-views.txt @@ -81,20 +81,20 @@ TemplateResponseMixin The response class to be returned by ``render_to_response`` method. Default is :class:`TemplateResponse `. - The template and context of TemplateResponse instances can be + The template and context of ``TemplateResponse`` instances can be altered later (e.g. in :ref:`template response middleware `). - Create TemplateResponse subclass and pass set it to - ``template_response_class`` if you need custom template loading or - custom context object instantiation. + If you need custom template loading or custom context object + instantiation, create a ``TemplateResponse`` subclass and assign it to + ``response_class``. .. method:: render_to_response(context, **response_kwargs) - Returns a ``self.template_response_class`` instance. + Returns a ``self.response_class`` instance. If any keyword arguments are provided, they will be - passed to the constructor of the response instance. + passed to the constructor of the response class. Calls :meth:`~TemplateResponseMixin.get_template_names()` to obtain the list of template names that will be searched looking for an existent diff --git a/docs/topics/class-based-views.txt b/docs/topics/class-based-views.txt index c90469b758d..a13f18993da 100644 --- a/docs/topics/class-based-views.txt +++ b/docs/topics/class-based-views.txt @@ -71,7 +71,7 @@ so we can just subclass it, and override the template name:: template_name = "about.html" Then, we just need to add this new view into our URLconf. As the class-based -views themselves are classes, we point the URL to the as_view class method +views themselves are classes, we point the URL to the ``as_view`` class method instead, which is the entry point for class-based views:: # urls.py @@ -83,7 +83,7 @@ instead, which is the entry point for class-based views:: ) Alternatively, if you're only changing a few simple attributes on a -class-based view, you can simply pass the new attributes into the as_view +class-based view, you can simply pass the new attributes into the ``as_view`` method call itself:: from django.conf.urls.defaults import * @@ -121,12 +121,12 @@ be using these models:: country = models.CharField(max_length=50) website = models.URLField() - def __unicode__(self): - return self.name - class Meta: ordering = ["-name"] + def __unicode__(self): + return self.name + class Book(models.Model): title = models.CharField(max_length=100) authors = models.ManyToManyField('Author') @@ -211,7 +211,7 @@ all the publishers in a variable named ``object_list``. While this works just fine, it isn't all that "friendly" to template authors: they have to "just know" that they're dealing with publishers here. -Well, if you're dealing with a Django object, this is already done for +Well, if you're dealing with a model object, this is already done for you. When you are dealing with an object or queryset, Django is able to populate the context using the verbose name (or the plural verbose name, in the case of a list of objects) of the object being displayed. @@ -353,7 +353,7 @@ key in the URL. Earlier we hard-coded the publisher's name in the URLconf, but what if we wanted to write a view that displayed all the books by some arbitrary publisher? -Handily, the ListView has a +Handily, the ``ListView`` has a :meth:`~django.views.generic.detail.ListView.get_queryset` method we can override. Previously, it has just been returning the value of the ``queryset`` attribute, but now we can add more logic. @@ -444,8 +444,8 @@ custom view: **(r'^authors/(?P\\d+)/$', AuthorDetailView.as_view()),** ) -Then we'd write our new view - ``get_object`` is the method that retrieves the -object, so we simply override it and wrap the call:: +Then we'd write our new view -- ``get_object`` is the method that retrieves the +object -- so we simply override it and wrap the call:: import datetime from books.models import Author @@ -473,7 +473,7 @@ object, so we simply override it and wrap the call:: .. note:: The URLconf here uses the named group ``pk`` - this name is the default - name that DetailView uses to find the value of the primary key used to + name that ``DetailView`` uses to find the value of the primary key used to filter the queryset. If you want to change it, you'll need to do your own ``get()`` call