Django DRF Обновление пользователя

Я пытаюсь создать API, используя Django-Rest-Framework (DRF) для 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)

1 ответ

Решение

По мнению, вы должны передать 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 метод (от GenericAPIView) получить правильный экземпляр; вместо того, чтобы повторно реализовать оба retrieve а также update,

Другие вопросы по тегам