Add view for viewing a newsletter post.

[Lab1.1]

Author: tim-schilling

project/newsletter/models.py

@@ -5,6 +5,7 @@
 from django.db import models
 from django.db.models import Exists, F, OuterRef
 from django.db.models.functions import Coalesce
+from django.urls import reverse
 from django.utils import timezone
 from django.utils.translation import gettext_lazy as _
 from martor.models import MartorField
@@ -159,7 +160,7 @@ def publish_date(self):
         return self.publish_at or self.created
 
     def get_absolute_url(self):
-        return f"/p/{self.slug}/"
+        return reverse("newsletter:view_post", kwargs={"lookup": self.slug})
 
 
 class SubscriptionQuerySet(models.QuerySet):

project/newsletter/test_views.py

@@ -94,6 +94,42 @@ def test_pagination(self):
         )
 
 
+class TestViewPost(DataTestCase):
+    def test_unauthenticated(self):
+        response = self.client.get(
+            reverse(
+                "newsletter:view_post", kwargs={"lookup": self.data.career_post.title}
+            )
+        )
+        self.assertTemplateUsed(response, "posts/detail.html")
+        self.assertEqual(response.context["post"], self.data.career_post)
+
+    def test_authenticated_private_post(self):
+        self.client.force_login(self.data.subscription.user)
+        response = self.client.get(
+            reverse(
+                "newsletter:view_post", kwargs={"lookup": self.data.private_post.title}
+            )
+        )
+        self.assertTemplateUsed(response, "posts/detail.html")
+        self.assertEqual(response.context["post"], self.data.private_post)
+
+    def test_mark_as_read(self):
+        notification = SubscriptionNotification.objects.create(
+            subscription=self.data.subscription,
+            post=self.data.all_post,
+            sent=timezone.now(),
+        )
+        self.client.force_login(self.data.subscription.user)
+        response = self.client.get(
+            reverse("newsletter:view_post", kwargs={"lookup": self.data.all_post.title})
+        )
+        self.assertTemplateUsed(response, "posts/detail.html")
+        self.assertEqual(response.context["post"], self.data.all_post)
+        notification.refresh_from_db()
+        self.assertIsNotNone(notification.read)
+
+
 class TestUnpublishedPosts(DataTestCase):
     url = reverse("newsletter:unpublished_posts")
 

project/newsletter/urls.py

@@ -21,6 +21,7 @@
         include(
             [
                 path("", views.list_posts, name="list_posts"),
+                path("<lookup>/", views.view_post, name="view_post"),
             ]
         ),
     ),

project/newsletter/views.py

@@ -9,16 +9,15 @@
 from django.core.files.base import ContentFile
 from django.core.files.storage import default_storage
 from django.core.paginator import Paginator
-from django.db.models import Case, F, Value, When
-from django.http import Http404
-from django.db.models import Count, Q
-from django.http import HttpResponse, JsonResponse
-from django.shortcuts import redirect, render, get_object_or_404
+from django.db.models import Case, Count, F, Q, Value, When
+from django.http import Http404, HttpResponse, JsonResponse
+from django.shortcuts import get_object_or_404, redirect, render
 from django.utils import timezone
 from django.utils.translation import gettext_lazy as _
 from django.views.decorators.http import require_http_methods
 from martor.utils import LazyEncoder
 
+from project.newsletter import operations
 from project.newsletter.forms import PostForm, SubscriptionForm
 from project.newsletter.models import Category, Post, Subscription
 
@@ -70,6 +69,27 @@ def list_posts(request):
     )
 
 
+@require_http_methods(["GET"])
+def view_post(request, lookup):
+    """
+    The post detail view.
+    """
+    posts = Post.objects.published().annotate_is_unread(request.user)
+    if not request.user.is_authenticated:
+        posts = posts.public()
+    post = posts.get(title=lookup)
+    if post.is_unread:
+        operations.mark_as_read(post, request.user)
+    return render(
+        request,
+        "posts/detail.html",
+        {
+            "post": post,
+            "open_graph_url": request.build_absolute_uri(post.get_absolute_url()),
+        },
+    )
+
+
 @require_http_methods(["GET", "POST"])
 @login_required()
 def update_subscription(request):