[Django]-Django: Integrity error UNIQUE constraint failed: user_profile.user_id

by

30👍

✅

It’s not strange. You already have a profile for that user, so adding another one breaks the unique constraint. You need to edit the existing one, not add a new one.

Also note that you’re not using the cleaned form data when you save, which you should be. Either use form.cleaned_data['bio'] etc, or even better just do form.save() which is the whole point of using a model form.

Putting that together:

try:
    profile = request.user.userprofile
except UserProfile.DoesNotExist:
    profile = UserProfile(user=request.user)

if request.method == 'POST':
    form = UserProfileForm(request.POST, instance=profile)
    if form.is_valid():
        form.save()
        return redirect...
else:
    form = UserProfileForm(instance=profile)
return render...
👤Daniel Roseman

9👍

I was getting the same error multiple times and I was really disheartened but finally came over the solution.

Convert:

user = models.OneToOneField(User)

to

user = models.ForeignKey(User)

This should solve the issue.

👤Shubham Gupta – TCH

2👍

Consider the case: If the same user makes the post request again on URL ‘profile/edit’ then your code will try to create the new UserProfile using the same user instance, but since this is a one to one field and you have already created one profile using that user hence it will throw integrity error.

So you should check first if the profile associated with that user already exists or not, then if it didn’t exist then create it.

👤localhost_3000

0👍

add (instance = request.user) in UserProfileForm(request.POST)

👤 0536

0👍

I accidentally got this error because I forgot to specify update_fields in my call to save. I had:

modified_instance.save(video_attr)

When I should have had:

modified_instance.save(update_fields=[video_attr])

The truthy string was being interpreted as the positional force_update parameter, which obviously causes problems if the record already exists.

👤Carcigenicate

-2👍

Yes you are making a OneToOne relation between User and UserProfile models that’s why you are facing integrity error,
for fixing this you can change it to ForeignKey
Like :

class UserProfile(models.Model):

user = models.ForeignKey(User,on_delete=models.CASCADE)
bio = models.TextField(blank=True)
...
...
👤vipin yadav

Leave a comment