Use 'source' to determine which fields should be autogenerated about djangorestframework-dataclasses

[oxan]

I'm not sure if suppressing the autogeneration of a field if another field uses it as source would be a sensible default. I can imagine usecases where source is used to add another representation, instead of replacing the default one all together, and it's also a breaking change. I'll have to check, but I also think that it'd be different behaviour from what DRF's built-in ModelSerializer does.

[Philipp-Userlike]

DRF built-in serializers are a bit different since they allow partial models by default. But when I set __all__ as fields I indeed get the same behaviour.

from django.db import models
from rest_framework import serializers


class MyModel(models.Model):
    id = models.AutoField(primary_key=True)


class MyModel2(models.Model):
    id = models.AutoField(primary_key=True)
    model = models.ForeignKey(MyModel, on_delete=models.CASCADE)


class MyModelSerializer(serializers.ModelSerializer):
    model_id = serializers.PrimaryKeyRelatedField(
        queryset=MyModel.objects.all(), source="model"
    )

    class Meta:
        model = MyModel2
        fields = "__all__"


MyModel(id=1).save()
MyModelSerializer(data={"model_id": 1}).is_valid(raise_exception=True)
# raises rest_framework.exceptions.ValidationError: {'model': [ErrorDetail(string='This field is required.', code='required')]}

I think probably it is more common that you have one representation of a field by 'source' and don't want to autogenerate another one, but seems hard to change now.