help me with Combine DetailView and DeleteView

Question

And, finally, we shouldn't let just anyone see the delete form. Wrap the form in the template with {% if user.is_authenticated %} so it'll only show for authenticated users.

articles/views.py

from django.contrib.auth.mixins import LoginRequiredMixin
from django.core.urlresolvers import reverse_lazy
from django.views import generic

from . import models


class ArticleList(generic.ListView):
    model = models.Article


class ArticleDetail(generic.DeleteView, generic.DetailView):
    model = models.Article
    template_name = 'articles/article_detail.html'


class ArticleCreate(LoginRequiredMixin, generic.CreateView):
    fields = ('title', 'body', 'author', 'published')
    model = models.Article


class ArticleUpdate(LoginRequiredMixin, generic.UpdateView):
    fields = ('title', 'body', 'author', 'published')
    model = models.Article


class ArticleDelete(LoginRequiredMixin, generic.DeleteView):
    model = models.Article
    success_url = reverse_lazy('articles:list')


class ArticleSearch(generic.ListView):
    model = models.Article

    def get_queryset(self):
        qs = super().get_queryset()
        term = self.kwargs.get('term')
        if term:
            return qs.filter(body__icontains=term)
        return qs.none()

articles/templates/articles/article_detail.html

<h1>{{ article.title }}</h1>
<p>By: {{ article.author }}</p>

{{ article.body|linebreaks }}

{% if user.is_authenticated %}
<hr>

<form method="post">
  {% csrf_token %} 
  {{ form }}
</form>

Answer 1 · 2019-03-12T14:38:53Z

March 12, 2019 2:38pm

You're very close! Just don't forget that in Django templates, if statements must be closed with an endif tag. Add that to the end of your template, and the challenge will pass.

Welcome to the Treehouse Community

Looking to learn something new?

Savious Munyaradzi Ngundu

Savious Munyaradzi Ngundu

help me with Combine DetailView and DeleteView

1 Answer

matth89

matth89