CODEWITHSUNDEEP
pythondjangodjango-rest-frameworkdjango-serializer
Melissa Stewart
Melissa Stewart

Reputation: 3635

Django rest framework image upload error

This is my UserProfile model,

class UserProfile(models.Model):
    user = models.OneToOneField(User, related_name='profile', on_delete=models.CASCADE, )
    badge = models.ImageField(upload_to='media/badges/', null=True)
    reputation = models.IntegerField(default=0)
    status = models.CharField(max_length=255, null=True, blank=True)
    thumbnail = models.ImageField(upload_to='media/user/', blank=True, null=True)

And this is the Serializer class,

class UserProfileSerializer(serializers.ModelSerializer):
    thumbnail = serializers.ImageField(max_length=None, use_url=True)

    class Meta:
        model = models.UserProfile
        fields = ('badge', 'reputation', 'status', 'thumbnail',)

My API to update an image looks like this,

class CreateUpdateUserThumbnail(views.APIView):

    def post(self, request, **kwargs):
        try:
            user = User.objects.get(id=kwargs.get('user_id'))
        except User.DoesNotExist:
            return Response({"Error": "User does not exist"}, status=status.HTTP_404_NOT_FOUND)
        request.data['user_id'] = user.id
        serializer = UserProfileSerializer(data=request.data)
        if serializer.is_valid():
            serializer.save()
            return Response(serializer.data, status=status.HTTP_201_CREATED)
        else:
            return Response(serializer.errors, status=status.HTTP_400_BAD_REQUEST)

This is the error that I get when I try to upload an image using POSTMAN.

return Database.Cursor.execute(self, query, params)
django.db.utils.IntegrityError: NOT NULL constraint failed: bouncer_userprofile.user_id

How do I associate a user_id with the serializer data?

Upvotes: 1

Views: 679

Answers (2)

Sardorbek Imomaliev
Sardorbek Imomaliev

Reputation: 15400

You need to pass user_profile instance on UserProfileSerializer instantiation

serializer = UserProfileSerializer(instance=user.profile, data=request.data)

Your view will look like this

class CreateUpdateUserThumbnail(views.APIView):

    def post(self, request, **kwargs):
        try:
            user = User.objects.get(id=kwargs.get('user_id'))
        except User.DoesNotExist:
            return Response({"Error": "User does not exist"}, status=status.HTTP_404_NOT_FOUND)
        serializer = UserProfileSerializer(instance=user.profile, data=request.data)
        if serializer.is_valid():
            serializer.save()
            return Response(serializer.data, status=status.HTTP_201_CREATED)
        else:
            return Response(serializer.errors, status=status.HTTP_400_BAD_REQUEST)

Upvotes: 1

user8060120
user8060120

Reputation:

the request.data is immutable so you should to create the copy first:

data = request.data.copy()
data['user_id'] = user.id
serializer = UserProfileSerializer(data=data)

Upvotes: 2

Related Questions