From 95b8d723472cef60cddce821687ccc05c72e4651 Mon Sep 17 00:00:00 2001 From: Peter Fogg Date: Thu, 30 Jul 2015 10:52:13 -0400 Subject: [PATCH] Revert "Remove unnecessary DB call in team pagination." This reverts commit 948f652cf0ae3b52d48c8a1289521856744a910f. --- lms/djangoapps/teams/views.py | 9 +-------- 1 file changed, 1 insertion(+), 8 deletions(-) diff --git a/lms/djangoapps/teams/views.py b/lms/djangoapps/teams/views.py index bd4d6b69de..1cd0edec7b 100644 --- a/lms/djangoapps/teams/views.py +++ b/lms/djangoapps/teams/views.py @@ -331,14 +331,7 @@ class TeamsListView(ExpandableFieldViewMixin, GenericAPIView): queryset = queryset.order_by(order_by_field) - # TODO: Remove this on update to Django 1.8 - # Use the cached length of the queryset in order to avoid - # making an extra database call to get the number of items in - # the collection - paginator = self.paginator_class(queryset, self.get_paginate_by()) - paginator._count = len(queryset) # pylint: disable=protected-access - page = paginator.page(int(request.QUERY_PARAMS.get('page', 1))) - # end TODO + page = self.paginate_queryset(queryset) serializer = self.get_pagination_serializer(page) serializer.context.update({'sort_order': order_by_input}) # pylint: disable=maybe-no-member return Response(serializer.data) # pylint: disable=maybe-no-member