Hi @loganknecht. Just as a sanity check, can you verify that your URL is correct? In your example URL, the query string starts with a / instead of a ?.

Ah @rpkilby - apologies, this was an oversight on my part from copying and pasting from Postman. The url is actually {{API_URL}}/api/v1/schools/?offset=49&course_name=Nihongo but I was removing the offset to be less confusing and succeeded in doing the opposite 😂

Your example doesn't raise any obvious issues to me. My best guess is that there is some issue with the API view. Have you added the DjangoFilterBackend to your settings/view's filter_backends? And have you set filterset_class = SchoolFilter (note that this used to be filter_class)?

Hey @rpkilby

Here is the view code I have

class SchoolViewSet(ViewSet):
    http_method_names = ["get", "post"]

    def list(self, request):
        all_school_models = SchoolModel.objects.all()

        filtered_school_models = SchoolFilter(request.GET, queryset=all_school_models)
        # import pdb
        # pdb.set_trace()

        paginator = HeaderLinkPagination()
        current_page_results = paginator.paginate_queryset(filtered_school_models.qs,
                                                           request)

        all_schools_serializer = SchoolSerializer(current_page_results,
                                                  many=True)
        response_data = {
            "schools": all_schools_serializer.data
        }

        response_to_return = paginator.get_paginated_response(response_data)
        return response_to_return

The pdb.set_trace() is where I posted the above interrogation.

I did not sett the DjangoFilterBackend as I assumed that was a global configuration and I'm only testing this on a single endpoint.

Additionally I did not set this for the class either for the same reason.

It is important to know that I CAN use this filter on the name of the SchoolModel but when I filter on the reverse foreign key school_course it does not work.

Gotcha - so you're creating the filterset in the view directly. In that case, yeah, you wouldn't need to set the filter backend/filterset class.

You might check to see if the data looks correct, if there were any errors, or if the SQL query is correctly formed. Try:

filtered_school_models = SchoolFilter(request.GET, queryset=all_school_models)
print(filtered_school_models.data)
print(filtered_school_models.errors)
print(filtered_school_models.qs)
print(str(filtered_school_models.qs.query))

@rpkilby Here is what I see

<QueryDict: {'offset': ['49'], 'course_name': ['Nihongo']}>

<QuerySet [<SchoolModel: SchoolModel object (51)>]>
SELECT "piano_gym_api_schoolmodel"."id", "piano_gym_api_schoolmodel"."name", "piano_gym_api_schoolmodel"."school_board_id" FROM "piano_gym_api_schoolmodel" INNER JOIN "piano_gym_api_schoolcoursemodel" ON ("piano_gym_api_schoolmodel"."id" = "piano_gym_api_schoolcoursemodel"."school_id") WHERE UPPER("piano_gym_api_schoolcoursemodel"."name"::text) LIKE UPPER(%Nihongo%) ORDER BY "piano_gym_api_schoolmodel"."id" ASC

@rpkilby This looks like it might be an issue with my paginator implementation which looks like this

class HeaderLinkPagination(LimitOffsetPagination):
    default_limit = settings.DEFAULT_LIMIT
    max_limit = settings.DEFAULT_MAX_LIMIT
    min_limit = settings.DEFAULT_MIN_LIMIT
    min_offset = settings.DEFAULT_MIN_OFFSET
    max_offset = settings.DEFAULT_MAX_OFFSET

    def get_paginated_response(self, data):
        next_url = self.get_next_link()
        previous_url = self.get_previous_link()

        links = []
        header_data = (
            (previous_url, "prev"),
            (next_url, "next"),
        )
        for url, label in header_data:
            if url is not None:
                links.append("<{}>; rel=\"{}\"".format(url, label))

        headers = {"Link": ", ".join(links)} if links else {}

        return Response(data, headers=headers)

    def paginate_queryset(self, queryset, request, view=None):

        limit = request.query_params.get("limit")
        offset = request.query_params.get("offset")

        if limit is None:
            limit = settings.DEFAULT_LIMIT

        if offset is None:
            offset = settings.DEFAULT_OFFSET

        limit = int(limit)
        if limit > self.max_limit:
            error_message = ("Limit should be less than or equal to {0}"
                             ).format(self.max_limit)
            errors = {"limit": [error_message]}
            raise ValidationError(errors)
        elif limit < self.min_limit:
            error_message = ("Limit should be greater than or equal to {0}"
                             ).format(self.min_limit)
            errors = {"limit": [error_message]}
            raise ValidationError(errors)

        offset = int(offset)
        if offset > self.max_offset:
            error_message = ("Offset should be less than or equal to {0}"
                             ).format(self.max_offset)
            errors = {"offset": [error_message]}
            raise ValidationError(errors)
        elif offset < self.min_offset:
            error_message = ("Offset should be greater than or equal to {0}"
                             ).format(self.min_offset)
            errors = {"offset": [error_message]}
            raise ValidationError(errors)
        import pdb
        pdb.set_trace()

        return super(self.__class__, self).paginate_queryset(queryset, request, view)

If you see a solution, please let me know. Stay tuned while I try to shake this out.

My guess is that this is related to the given offset. There is only 1 object in the filtered queryset, but the request is telling the paginator to skip the first 49 items. If you were to check

print(filtered_school_models[49:])

I assume you'd get <QuerySet []>.

Closing, as it looks like this is a pagination issue.

Hey @rpkilby

I just want to say thank you so much for talking me through this. This wasn't even a django-filter problem. I was just being a dingus. You rubber ducking me is super super appreciated.

You are amazing, and once again, thank you for this amazing library!

No worries. Happy to help!