'Topic authentication', section 'Storing additional information about users' there is bug in the code

[supreet.sethi@…]

In the example code for creating profile using post_save event of User. The code is written thus

def create_user_profile(sender, instance, created, **kwargs):
    if created:
        UserProfile.objects.create(user=instance)

Whereas it should be thus:

def create_user_profile(sender, instance, created, **kwargs):
    if created:
        UserProfile.objects.create(user=instance).save()

Otherwise it generates "IntegrityError: column user_id is not unique" error.

[Karen Tracey]

The proposed change is unnecessary (create() already saves the new object to the DB) and would not do anything to fix an IntegrityError. You'd get an IntegrityError if somehow this create_user_profile was called with create set to True for some user instance where there was already a UserProfile object for it in the DB. I'm not sure how that would have happened, but tacking .save() onto that existing line of code wouldn't fix that problem.

[bwanamarko@…]

as stated in the Status comment by @kmtracy m it raises the integrity exception because the UserProfile is already created changing the create to get_or_create solves the problem.

def create_user_profile(sender, instance, created, **kwargs):
    if created:
        UserProfile.objects.get_or_create(user=instance)