And, finally, we shouldn't let just anyone see the delete form...

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.

Not sure how to finish this code challenge. Task 3 of 3.

articles/views.py

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(generic.CreateView):
    fields = ('title', 'body', 'author', 'published')
    model = models.Article


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


class ArticleDelete(generic.DeleteView):
    model = models.Article


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>

  {% endif %}

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

Answer 1 · 2021-05-15T17:18:13Z

May 15, 2021 5:18pm

For article_detail.html: If you move your form into your if-statement it will pass.

Welcome to the Treehouse Community

Looking to learn something new?

Chris Couch

Chris Couch

And, finally, we shouldn't let just anyone see the delete form...

1 Answer

Tom Nguyen

Tom Nguyen