Django изменить профиль пользователя

Я пытаюсь создать форму "Изменить профиль" на фронте. Случается, что моя форма (я не уверен на 100%) пытается создать пользователя вместо того, чтобы находить текущего пользователя и обновлять его профиль. Поэтому я думаю, что проблема. Здесь было много вопросов, но ни один из них не был достаточно ясным. Поля, которые я пытаюсь изменить, - это электронная почта, имя и фамилия. (Также я хотел бы добавить uda

forms.py

class UpdateProfile(forms.ModelForm):
    username = forms.CharField(required=True)
    email = forms.EmailField(required=True)
    first_name = forms.CharField(required=False)
    last_name = forms.CharField(required=False)

    class Meta:
        model = User
        fields = ('username', 'email', 'first_name', 'last_name')

    def clean_email(self):
        username = self.cleaned_data.get('username')
        email = self.cleaned_data.get('email')

        if email and User.objects.filter(email=email).exclude(username=username).count():
            raise forms.ValidationError('This email address is already in use. Please supply a different email address.')
        return email

    def save(self, commit=True):
        user = super(RegistrationForm, self).save(commit=False)
        user.email = self.cleaned_data['email']

        if commit:
            user.save()

        return user

views.py

def update_profile(request):
    args = {}

    if request.method == 'POST':
        form = UpdateProfile(request.POST)
        form.actual_user = request.user
        if form.is_valid():
            form.save()
            return HttpResponseRedirect(reverse('update_profile_success'))
    else:
        form = UpdateProfile()

    args['form'] = form
    return render(request, 'registration/update_profile.html', args)

Ответ 1

Вы очень близки. Когда вы создаете форму, вам необходимо передать объект User, который вы изменяете как аргумент instance.

Из документов:

Подкласс ModelForm может принимать существующий экземпляр модели как ключевое слово instance; если это будет предоставлено, save() обновит этот экземпляр.

В вашем коде это будет выглядеть так:

form = UpdateProfile(request.POST, instance=request.user)
if form.is_valid():
    ...

Вы можете посмотреть дополнительную информацию здесь: https://docs.djangoproject.com/en/1.6/topics/forms/modelforms/#the-save-method

Ответ 2

Или, если вы хотите перейти на классный взгляд, используя UpdateView, вы можете сделать что-то вроде этого в views.py:

class UpdateProfile(UpdateView):
    model = MyProfile
    fields = ['first_name', 'last_name', 'image', 'url', 'biography', '...'] # Keep listing whatever fields 
    # the combined UserProfile and User exposes.
    template_name = 'user_update.html'
    slug_field = 'username'
    slug_url_kwarg = 'slug'

где у вас есть что-то вроде этого для MyProfile в models.py:

class MyProfile(AbstractUser):
    image = models.ImageField(upload_to='uploads/profile/')
    url = models.URLField()
    biography = models.CharField(max_length=1000)

и ваш urls.py, как это (предполагая, что вы используете django allauth и хотите почитать соглашение с URL):

....
url(r'^accounts/update/(?P<slug>[\-\w]+)/$', views.UpdateProfile.as_view(), name='update_user'),
....

Остальное - удовольствие от Django! Я рекомендую вам писать меньше кода, если бы вы могли выполнять основные задачи CRUD, если только не нужно делать что-то обычай, из которых даже вы все равно можете уйти, расширяя представления на основе классов.

И в случае, если вы заходите сюда, см. здесь: Django Docs на основе классов