使用Django rest更新ManyToMany字段

Question

我正在尝试设置此API，以便可以使用“ PUT”更新模型“ MOVIE”中某个项目上的一个/多个“ TAG”。标签是MOVIE上的M2M。我正在电影中该物品的PK上发布。

我的httpie工作（返回200OK），但未创建任何内容。当我发布整个JSON（使用访存）时，它只是创建TAG，但在MOVIE（链接）上没有M2M关系。

httpie

http -f PUT http://localhost:8000/api/Edit/3/ tag:='{"name": "TEST"}'

型号

class Tag(models.Model):
    name = models.CharField("Name", max_length=5000, blank=True)
    taglevel = models.IntegerField("Tag level", null=True, blank=True)

class Movie(models.Model):
    title = models.CharField("Whats happening?", max_length=10000, blank=True)
    tag = models.ManyToManyField('Tag', blank=True)

Serializers.py

class Tag1Serializer(serializers.ModelSerializer):
    class Meta:
        model = Tag
        fields = ('name',)

class EditSerializer(serializers.ModelSerializer):
    tag = Tag1Serializer(many=True, read_only=True)
    class Meta:
            model = Movie
            fields = ('title', 'tag', 'info', 'created',  'status')

    def update(self, instance, validated_data):
        import pdb; pdb.set_trace()
        tags_data = validated_data.pop('tag')
        for tag_data in tags_data:
            tag_qs = Tag.objects.filter(name__iexact=tag_data['name'])
            if tag_qs.exists():
                tag = tag_qs.first()
            else:
                tag = Tag.objects.get(**tag_data)
            instance.tag.add(tag)
        return movie

Views.py

class MovieViewSet(viewsets.ModelViewSet):
    queryset = Movie.objects.all()
    serializer_class = MovieSerializer

错误：

Traceback
    tags_data = validated_data.pop('tag')
KeyError: 'tag'

Answer 1

putdrf模型序列化器类上没有任何方法，因此没有任何调用put(self, validated_data)。使用：update(self, instance, validated_data)代替。有关保存实例的文档：http : //www.django-rest-framework.org/api-guide/serializers/#saving-instances

django模型queryset也没有：Movie.objects.put和Tag.objects.put。您已经有instance电影的参数，如果要查询标签，也许需要Tag.objects.get或Tag.objects.filter？QuerySet API参考：https : //docs.djangoproject.com/en/1.10/ref/models/querysets/#queryset-api

确认调用了序列化程序方法之后，也许您应该使用drf test api客户端为其编写测试，以便能够轻松发现错误：http : //www.django-rest-framework.org/api-guide/testing/ #apiclient

serializers.py

class TagSerializer(serializers.ModelSerializer):
    class Meta:
        model = Tag
        fields = ('name', 'taglevel', 'id')


class MovieSerializer(serializers.ModelSerializer):
    tag = TagSerializer(many=True, read_only=False)

    class Meta:
        model = Movie
        ordering = ('-created',)
        fields = ('title', 'pk', 'tag')

    def update(self, instance, validated_data):
        tags_data = validated_data.pop('tag')
        instance = super(MovieSerializer, self).update(instance, validated_data)

        for tag_data in tags_data:
            tag_qs = Tag.objects.filter(name__iexact=tag_data['name'])

            if tag_qs.exists():
                tag = tag_qs.first()
            else:
                tag = Tag.objects.create(**tag_data)

            instance.tag.add(tag)

        return instance

tests.py

class TestMovies(TestCase):
    def test_movies(self):
        movie = Movie.objects.create(title='original title')

        client = APIClient()
        response = client.put('/movies/{}/'.format(movie.id), {
            'title': 'TEST title',
            'tag': [
                {'name': 'Test item', 'taglevel': 1}
            ]
        }, format='json')

        self.assertEqual(response.status_code, 200, response.content)
        # ...add more specific asserts

Answer 2

好的。我答应等我弄清楚后就回来。这可能不是完全数据安全的，因为 django 尚未验证传入的数据，因此我在对 python 和 django 相对无知的情况下做出了一些假设。如果有比我聪明的人可以扩展这个答案，请联系我。

注意：我坚定地遵守编写软件的干净代码标准。多年来它对我很有帮助。我知道它不是 Python 代码的元，但如果没有小的、紧密关注的方法，它会感觉很草率。

视图.py

如果您不能有重复的内容，则必须先自行清除相关对象，然后才能添加新对象。这是我能找到的针对我的用例可靠地删除 m2m 的唯一方法。我需要确保没有重复项，并且我期望一个原子模型。你的旅费可能会改变。

class MovieViewSet(viewsets.ModelViewSet):
    queryset = Movie.objects.all()
    serializer_class = MovieSerializer

    def update(self, requiest, *args, **kwargs):
        movie = self.get_object()
        movie.tags.clear()
        return super().update(request, *args, **kwargs)

序列化器.py

您必须挂钩to_internal_value序列化器方法才能获取所需的数据，因为验证器会忽略 m2m 字段。

class Tag1Serializer(serializers.ModelSerializer):
    class Meta:
        model = Tag
        fields = ('name',)

class EditSerializer(serializers.ModelSerializer):
    tag = Tag1Serializer(many=True, read_only=True)
    class Meta:
        model = Movie
        fields = ('title', 'tag', 'info', 'created',  'status')

    def to_internal_value(self, data):
        movie_id = data.get('id')
        #if it's new, we can safely assume there's no related objects.
        #you can skip this bit if you can't make that assumption.
        if self.check_is_new_movie(movie_id):
            return super().to_internal_value(data)
        #it's not new, handle relations and then let the default do its thing
        self.save_data(movie_id, data)
        return super().to_internal_value(data)

    def check_is_new_movie(self, movie_id):
        return not movie_id

    def save_data(self, movie_id, data):
        movie = Movie.objects.filter(id=movie_id).first()
        #the data we have is raw json (string).  Listify converts it to python primitives.
        tags_data = Utils.listify(data.get('tags'))

        for tag_data in tags_data:
            tag_qs = Tag.objects.filter(name__iexact=tag_data['name'])
            #I am also assuming that the tag already exists.  
            #If it doesn't, you have to handle that.
            if tag_qs.exists():
                tag = tag_qs.first()
                movie.tags.add(tag)

实用程序.py

from types import *
class Utils:
#python treats strings as iterables; this utility casts a string as a list and ignores iterables
def listify(arg):
    if Utils.is_sequence(arg) and not isinstance(arg, dict):
        return arg
    return [arg,]

 def is_sequence(arg):
     if isinstance(arg, str):
         return False
     if hasattr(arg, "__iter__"):
         return True

测试.py

根据需要调整网址以使其发挥作用。逻辑应该是正确的，但可能需要一些调整才能正确反映您的模型和序列化器。它更复杂，因为我们必须为 APIClient 创建 json 数据以与 put 请求一起发送。

class MovieAPITest(APITestCase):
    def setUp(self):
        self.url = '/movies/'

    def test_add_tag(self):
        movie = Movie.objects.create(name="add_tag_movie")
        tag = Tag.objects.create(name="add_tag")
        movie_id = str(movie.id)
        url = self.url + movie_id + '/'

        data = EditSerializer(movie).data
        data.update({'tags': Tag1Serializer(tag).data})
        json_data = json.dumps(data)

        self.client.put(url, json_data, content_type='application/json')
        self.assertEqual(movie.tags.count(), 1)