[PATCH] REST: squish final open-codings of get_object_or_404
Daniel Axtens
dja at axtens.net
Mon Aug 23 17:47:00 AEST 2021
Basically, finish the job of commit fecf7c86c2c5 ("urls: Add missing path
converters for REST APIs"). With this commit, there are no more uses of
Http404 in patchwork/api
Signed-off-by: Daniel Axtens <dja at axtens.net>
---
patchwork/api/comment.py | 8 +++-----
1 file changed, 3 insertions(+), 5 deletions(-)
diff --git a/patchwork/api/comment.py b/patchwork/api/comment.py
index 0c578b44f1cb..e9c52b990080 100644
--- a/patchwork/api/comment.py
+++ b/patchwork/api/comment.py
@@ -5,7 +5,7 @@
import email.parser
-from django.http import Http404
+from rest_framework.generics import get_object_or_404
from rest_framework.generics import ListAPIView
from rest_framework.serializers import SerializerMethodField
@@ -86,8 +86,7 @@ class CoverCommentList(ListAPIView):
lookup_url_kwarg = 'pk'
def get_queryset(self):
- if not Cover.objects.filter(pk=self.kwargs['pk']).exists():
- raise Http404
+ get_object_or_404(Cover, pk=self.kwargs['pk'])
return CoverComment.objects.filter(
cover=self.kwargs['pk']
@@ -105,8 +104,7 @@ class PatchCommentList(ListAPIView):
lookup_url_kwarg = 'patch_id'
def get_queryset(self):
- if not Patch.objects.filter(id=self.kwargs['patch_id']).exists():
- raise Http404
+ get_object_or_404(Patch, id=self.kwargs['patch_id'])
return PatchComment.objects.filter(
patch=self.kwargs['patch_id']
--
2.30.2
More information about the Patchwork
mailing list