问题
So I've extended my user with the field score
like this:
models.py:
class UserProfile(models.Model):
user = models.OneToOneField(User, on_delete=models.CASCADE)
score = models.IntegerField(default=0);
which seemed to be along the lines of the recommended way.
Then I tried to access the user's userprofile
in my views:
views.py:
player = request.user.userprofile
which seemed to align with the recommended way as well, but that is where I get my error:
RelatedObjectDoesNotExist
User has no userprofile.
If I change userprofile
to something else I get another error:
AttributeError
'User' object has no attribute 'flowerpot'
When I try the following code:
print request.user
print UserProfile.objects.all()
I get the console output:
post_test1
[]
EDIT
I have two superusers, seven users I created before extending the user, and one user (post_test1) that I created after extending the user.
EDIT 2
It seams clear that what I need is to create a post_save
handler that creates a new profile whenever the User object is created.
This seemed simple enough when I read it, went to the page that was linked to, which was a list of all the signals Django sends. I looked up post_save
and it said:
Like pre_save, but sent at the end of the save() method.
Alright, so I look up pre_save
and it says:
This is sent at the beginning of a model’s save() method.
I've interpreted it like this: When I create my user (in my views.py
) the save()
method should be called, which hasn't been the case up until now, and after that a post_save
should be sent(?), which will create a new profile whenever the User object is created!
So now I'm ready to start looking at examples, so I google:
django post save example
Here it looks like I'm supposed to add something that looks like a decorator @receiver(post_save, ...
Here it looks like I'm supposed to alter multiple files and write a signal definition?
This one also seems to imply multiple files (including a signals.py
)
It looks like there's a lot more to it than I first thought. Could anyone here either explain how I'm to do this or show me to some good resources on how signals work?
Right now my create_user
view look like this:
def create_user(request):
if request.method == "POST":
form = UserCreationForm(request.POST)
if form.is_valid():
username = form.cleaned_data["username"]
password = form.cleaned_data["password1"]
new_user = User.objects.create_user(username=username, password=password)
return redirect('play')
else:
form = UserCreationForm()
return render(request, 'antonymapp/create_user.html', {'form': form})
Should I call new_user.save()
before returning? If yes, why has it worked up until now? I have a bunch of users that I've created while testing this view. It also looks like somewhere around here post_save.connect(create_profile, sender=User)
should be added?
回答1:
You have to create a userprofile
for the user first:
profile = UserProfile.objects.create(user=request.user)
In your views.py you can use get_or_create
so that a userprofile is created for a user if the user doesn't have one.
player, created = UserProfile.objects.get_or_create(user=request.user)
UPDATE: For automatically creating user profiles every time a new user is made, use singals. In myapp/signals.py
do something like this:
@receiver(post_save, sender=User, dispatch_uid='save_new_user_profile')
def save_profile(sender, instance, created, **kwargs):
user = instance
if created:
profile = UserProfile(user=user)
profile.save()
回答2:
Nothing in what you've done forces the creation of a UserProfile
object when a User
is created. There are two basic ways of handling this:
If you always want a
UserProfile
to exist (which seems like the case as you give adefault
value toscore
, create a post_save handler that creates a new profile when ever theUser
object is created (but not every time it's saved, so make sure to check thecreated
argument in the handler).If it's expected a user may not have a profile, you need to catch the
UserProfile.DoesNotExist
exception when trying to access it. If you do this frequently, make some kind of helper function.
UPDATED TO ANSWER SIGNAL QUESTION
It also looks like somewhere around here
post_save.connect(create_profile, sender=User)
should be added?
You would need to define a function called create_profile
and then wire it up as you have shown. I typically do this right in the models.py
file that includes the sender
but in this case where the sender is a built-in Django model and you're already importing that model into the file where you define your UserProfile
that's the place to do it. It would look something like:
def create_profile(sender, instance, created, *args, **kwargs):
# ignore if this is an existing User
if not created:
return
UserProfile.objects.create(user=instance)
post_save.connect(create_profile, sender=User)
回答3:
Answering your updated question. There is already a save() method in models and it gets called every time you save a model (in your case when you call create_user()). So all you need to do is define a handler function and connect it to the post_save signal.
def create_extension(sender, instance, created, *args, **kwargs): if created: # do your thing models.signals.post_save.connect(create_extension, sender=User, dispatch_uid='create_extension')
I usually put the handler functions into a separate file signals.py and add connect() statements at the end of models.py
回答4:
If you are getting this error even if you've tried the suggestions above, it may caused by the fact that the first user you had created (with createsuperuser command) does not have a profile.
I was getting this error when I tried to login with that user. I solved it this way:
-Create a new user.
-Undo the changes. (Erase the code you've written for Profile or make them comment lines)
-Log in to your superuser.
-Give admin authorization to newly created user.
Now you can delete the first user. (The user with no profile)
来源:https://stackoverflow.com/questions/36317816/relatedobjectdoesnotexist-user-has-no-userprofile