5

我正在尝试使用 Django-Rest-Framework (DRF) 创建一个 api 来对用户进行 CRUD。我设法创建并读取了一个用户，但由于某种原因更新不起作用。似乎它正在尝试创建用户而不是更新它，因为它以“用户名已存在”错误消息进行响应。当我尝试传递尚未在数据库中的电子邮件时，它只会创建一个新用户。有谁知道为什么会这样？

这是我的用户序列化器：

class UserSerializer(FriendlyErrorMessagesMixin, serializers.ModelSerializer):
    password = serializers.CharField(
        max_length=128,
        min_length=8,
        write_only=True
    )

    class Meta:
        model = User
        fields = ('username',
                  'email',
                  'password',
                  'name',
                  'house_number',
                  'address_line_one',
                  'address_line_two',
                  'address_line_three',
                  'city',
                  'postcode',
                  'phone_number')
        read_only_fields = ('date_created', 'date_modified', 'username')

    def update(self, instance, validated_data):

        password = validated_data.pop('password', None)

        for (key, value) in validated_data.items():
            setattr(instance, key, value)

        if password is not None:
            instance.set_password(password)

        instance.save()

        return instance

这是我的观点：

class UserRetrieveUpdateAPIView(RetrieveUpdateAPIView):
    permission_classes = (IsAuthenticated,)
    serializer_class = UserSerializer

    def retrieve(self, request, *args, **kwargs):
        serializer = self.serializer_class(request.user)
        return Response(serializer.data, status=status.HTTP_200_OK)

    def update(self, request, *args, **kwargs):
        serializer = self.serializer_class(data=request.data, partial=True)
        serializer.is_valid(raise_exception=True)
        serializer.save()

        return Response(serializer.data, status=status.HTTP_200_OK)
4

1 回答 1

12

在视图中，您应该将Serializer要更新的实例作为第一个参数传递，否则将创建一个新对象。

要更新经过身份验证的用户，您将执行以下操作：

def update(self, request, *args, **kwargs):
    serializer = self.serializer_class(request.user, data=request.data, partial=True)
    serializer.is_valid(raise_exception=True)
    serializer.save()
    return Response(serializer.data, status=status.HTTP_200_OK)

通常，您可能希望使用或覆盖get_object方法 (from GenericAPIView) 来检索正确的实例；而不是重新实现retrieveand update。

于 2017-07-16T12:53:09.717 回答