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、正しいインスタンスを取得することをお勧めします。retrieveと の両方を再実装する代わりにupdate。

于 2017-07-16T12:53:09.717 に答える