Passing context to serializer with `request` param causes `AttributeError` on `query_params` about drf-flex-fields

[sevdog]

What is the point of using a DRF serialier with a standard request?

I mean: you should know when you are in an APIView (or @api_view) or not, if not you should take care of providing the right argument to the serializer.

[gitaarik]

In my case, I was using the serializer outside of a DRF API view. I like to re-use serializers sometimes for functionality outside the API.

[sevdog]

In that case the "fix" is simple:

from rest_framwork.request import Request


...
MySerializer(data, context={'request': Request(request)})

Since you are using this outside rest framework "middleware" I belive it is normal that you should provide the appropriated changes.

[gitaarik]

Aha, fair enough. Although usually serializers work fine with Django request objects. Just not when using FlexFields. So when you're introducing FlexFieldsModelSerializer or FlexFieldsSerializerMixin in your code, you can suddenly have this issue, and you would need to figure out what the problem is and apply the fix to potentially multiple spots in your code. People might think that FlexFields just doesn't work for their use-case, or with their version of Django / DRF.

But I understand your reasoning as well. We could also check if query_params exists, and if not, raise a custom exception with a descriptive error message so the user understands it should wrap the Django Request object in a Rest Framework Request object.

[sevdog]

To solve that the easiest fix would be to make flex-fields use request.GET instead of request.query_params (which is just an alias property).

[gitaarik]

Aha yeah that sounds like a good idea. Then it doesn't have a dependency on DRF request objects, and it can just work with common Django request objects.