Как правильно переопределить get_queryset в modelviewset без нарушения запроса по pk?

У меня есть ModelViewSet:

class OrderViewSet(viewsets.ModelViewSet):
serializer_class = OrderSerializer
pagination_class = DefaultPagination
queryset = Order.objects.all()

def get_queryset(self):
    userId = self.request.query_params.get('userId')
    if userId is not None:
        query = Order.objects.filter(owner__id=userId).order_by('-createdAt')
    else:
        query = Order.objects.order_by('-createdAt')
    return query

Здесь url регистрации

router = routers.DefaultRouter()
router.register('api/v1/order', OrderViewSet, basename='order')

Я обнаружил, что после переопределения get_queryset я больше не могу делать запрос по одному идентификатору заказа, как показано ниже: /api/v1/order/1, он просто возвращает detail: not found

Я вижу, что django говорит, что он поддерживает:

^api/v1/order/$ [name='order-list']
^api/v1/order\.(?P<format>[a-z0-9]+)/?$ [name='order-list']
^api/v1/order/(?P<pk>[^/.]+)/$ [name='order-detail']
^api/v1/order/(?P<pk>[^/.]+)\.(?P<format>[a-z0-9]+)/?$ [name='order-detail']

каким должен быть правильный способ сделать это? Спасибо!

Как насчет использования представлений Django, основанных на функциях и классах, в своих интересах. Я могу предложить решение, в котором вы можете создать еще одно представление на основе функций для обработки вышеуказанной функциональности и позволить методу get представления на основе классов обрабатывать набор запросов pk.

Чтобы исправить это, вызовите super() при переопределении get_queryset()

Попробуйте

def get_queryset(self):
    queryset = super(OrderViewSet, self).get_queryset()

    userId = self.request.query_params.get('userId')
    if userId is not None:
         queryset = queryset.filter(owner__id=userId).order_by('-createdAt')
    else:
        queryset = queryset.order_by('-createdAt')
    return query
Вернуться на верх