Skip to content

Commit fd2c07b

Browse files
authored
Merge pull request #5803 from rtfd/davidfischer/apiv3-skip-expensive-all-query
Optimize requests to APIv3
2 parents ac6bccf + 723cd8a commit fd2c07b

File tree

1 file changed

+2
-7
lines changed

1 file changed

+2
-7
lines changed

readthedocs/api/v3/mixins.py

Lines changed: 2 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -81,17 +81,12 @@ def get_queryset(self):
8181
4. raise a ``NotFound`` exception otherwise
8282
"""
8383

84-
# NOTE: ``super().get_queryset`` produces the filter by ``NestedViewSetMixin``
85-
# we need to have defined the class attribute as ``queryset = Model.objects.all()``
84+
# We need to have defined the class attribute as ``queryset = Model.objects.all()``
8685
queryset = super().get_queryset()
8786

8887
# Detail requests are public
8988
if self.detail:
9089
return self.detail_objects(queryset, self.request.user)
9190

9291
# List view are only allowed if user is owner of parent project
93-
listing_objects = self.listing_objects(queryset, self.request.user)
94-
if listing_objects:
95-
return listing_objects
96-
97-
raise NotFound
92+
return self.listing_objects(queryset, self.request.user)

0 commit comments

Comments
 (0)