Changing LogView to DetailView, as per Volcy's review

1 files changed, 4 insertions, 26 deletions

diff --git a/pydis_site/apps/staff/viewsets/logs.py b/pydis_site/apps/staff/viewsets/logs.py
index 0898d606..22dede95 100644
--- a/pydis_site/apps/staff/viewsets/logs.py
+++ b/pydis_site/apps/staff/viewsets/logs.py
@@ -1,33 +1,11 @@
-import logging
-
-from django.core.handlers.wsgi import WSGIRequest
-from django.http import HttpResponse
-from django.shortcuts import get_object_or_404, render
-from django.views import View
+from django.views.generic.detail import DetailView
 
 from pydis_site.apps.api.models.bot.message_deletion_context import MessageDeletionContext
 
-log = logging.getLogger(__name__)
-
 
-class LogView(View):
+class LogView(DetailView):
     """The default view for the Deleted Messages logs."""
 
+    model = MessageDeletionContext
+    context_object_name = "deletion_context"
     template_name = "staff/logs.html"
-
-    def get(self, request: WSGIRequest, pk: int) -> HttpResponse:
-        """Get method that answers a request with an html response by rendering a template."""
-        message_context = get_object_or_404(MessageDeletionContext, pk=pk)
-
-        actor = message_context.actor
-        creation = message_context.creation
-        messages = message_context.deletedmessage_set.all()
-
-        template_fields = {
-            'actor': actor,
-            'actor_colour': message_context.actor.top_role.colour,
-            'creation': creation,
-            'messages': messages
-        }
-
-        return render(request, self.template_name, template_fields)