Не могу заполнить связную таблицу в djangoю. Что не так?

Question

[justdani]

Ребят, бьюсь который день уже. Никак не получется наполнить связную модель в шаблоне. ЗАполняется модель User, но не заполняется модель Band.
models.py

from django.db import models
from django.contrib.auth.models import User


class Band(models.Model):
    user = models.OneToOneField(User, on_delete=models.CASCADE, default="")
    band_name = models.CharField(max_length=50)
    phone = models.CharField(max_length=15)
    vk_page = models.URLField(blank=True, null=True)


class Repetition(models.Model):
    band = models.ForeignKey(Band, on_delete=models.CASCADE)
    start_time = models.CharField(max_length=5)
    end_time = models.CharField(max_length=5)
    date = models.DateField()


def user_post_save(sender, instance,**kwargs):
    (profile, new) = Band.objects.get_or_create(user=instance)


def save_user_profile(sender, instance, **kwargs):
    instance.band.save()


models.signals.post_save.connect(user_post_save, sender=User)
models.signals.post_save.connect(save_user_profile, sender=User)

forms.py

from django import forms
from .models import User, Repetition, Band


class UserForm(forms.ModelForm):
    class Meta:
        model = User
        fields = ('username', 'password')


class BandForm(forms.ModelForm):
    class Meta:
        model = Band
        fields = ('band_name', 'phone', 'vk_page',)
    password = forms.CharField(widget=forms.PasswordInput, label="Пароль")


class RepetitionForm(forms.ModelForm):
    class Meta:
        model = Repetition
        fields = ('start_time', 'end_time', 'date')

views.py

from django.contrib import messages
from django.shortcuts import render, redirect
from .forms import BandForm, RepetitionForm
from .models import Band, Repetition, User


def band_form(request):
    user = User()
    if request.method == 'POST':
        _band_form = BandForm(request.POST, instance=user)
        _repetition_form = RepetitionForm(request.POST)
        if _band_form.is_valid():
            data = _band_form.cleaned_data          
            user.username = data['band_name']
            user.set_password(data['password'])
            _band_form.save()
            return redirect('/')
        else:
            messages.error(request, 'Whoo!')
    else:
        _band_form = BandForm(instance=user)
        _repetition_form = RepetitionForm()
    return render(request, 'auth.html', {'band_form': _band_form, 'repetition_form': _repetition_form})

Answer 1

[Ростислав Григорьев]

Потому что:

1. _band_form = BandForm(request.POST, instance=user) - здесь задается instance модели User, а нужен Band.
   
2. Если инстанса Band еще нет, его не нужно передавать
   
3. В мета у формы BandForm для fields не указано поле user, значит сохраняться оно не будет даже если передать такой параметр.
   
4. user = User()- несохраненный инстанс юзера бесполезно указывать как ForeignKey для Band. Необходимо его самого сначала сохранить
   

Comments

[justdani]

В мета у формы BandForm для fields не указано поле user, значит сохраняться оно не будет даже если передать такой параметр

Так поле уникальное же. Если его указать в fields оно рендерится и не даёт вообще ничего создать. После того как я убрал создание экземпляра band.

from django.contrib import messages
from django.shortcuts import render, redirect
from .forms import BandForm, RepetitionForm
from .models import Band, Repetition, User


def band_form(request):
    if request.method == 'POST':
        _band_form = BandForm(request.POST)
        _repetition_form = RepetitionForm(request.POST)
        if _band_form.is_valid():
            _band_form.save()
            return redirect('/')
        else:
            messages.error(request, 'Whoo!')
    else:
        _band_form = BandForm()
        _repetition_form = RepetitionForm()
    return render(request, 'auth.html', {'band_form': _band_form, 'repetition_form': _repetition_form})

from django import forms
from .models import User, Repetition, Band


class UserForm(forms.ModelForm):
    class Meta:
        model = User
        fields = ('username', 'password')


class BandForm(forms.ModelForm):
    class Meta:
        model = Band
        fields = ('band_name', 'phone', 'vk_page', 'user')
    password = forms.CharField(widget=forms.PasswordInput, label="Пароль")


class RepetitionForm(forms.ModelForm):
    class Meta:
        model = Repetition
        fields = ('start_time', 'end_time', 'date')

[justdani]

Не до конца понимаю как передать экземпляр одной модели в другую.

[Ростислав Григорьев]

Так поле уникальное же

Оно не уникальное. У каждой записи Band может быть один User. Для того чтобы сохранить в band юзера, нужно сначала его создать. Каким образом - это уже другой вопрос. Можно например через админку или добавить форму на другой странице, или сделать отдельную функцию, вариантов масса. После того как будет юзер - отправить его айдишник в эту вьюху параметром user - и все будет.

[justdani]

Ростислав Григорьев, Спасибо большое

[justdani]

Ростислав Григорьев, ещё вопрос, а как во вьюху передать вообще параметром (из отдельной функции), для этого же она должна где то вызываться?

[justdani]

Ростислав Григорьев, У меня ещё проблема в том, что модель User расширяется моделью Band и заполняются формы в одном шаблоне. Как сделать создание User отдельной функцией если post запрос один со всеми данными сразу? Может тогда вызвать функцию внутри вьюхи и передать в неё необходимые для user параметры, а возвращать id и подставлять id в user который onetoonefield у Band?

[justdani]

Ростислав Григорьев, Попробовал так, не заработало

from django.contrib import messages
from django.shortcuts import render, redirect
from .forms import BandForm, RepetitionForm
from .models import Band, Repetition, User


def band_form(request):
    if request.method == 'POST':
        user = User()
        band = Band()
        _band_form = BandForm(request.POST)
        # _repetition_form = RepetitionForm(request.POST)
        if _band_form.is_valid():
            data = _band_form.cleaned_data
            band_name = data['band_name']
            password = data['password']
            idi = user_reg(band_name, password, user)
            band.user = idi
            band.phone = data['phone']
            band.save()
            return redirect('/')
        else:
            messages.error(request, 'Whoo!')
    else:
        _band_form = BandForm()
        _repetition_form = RepetitionForm()
    return render(request, 'auth.html', {'band_form': _band_form})


def user_reg(username, password, user):
    user.username = username
    user.set_password(password)
    user.save()
    return user

[justdani]

Ростислав Григорьев, Сделал так, всё заработало

from django.contrib import messages
from django.shortcuts import render, redirect
from .forms import BandForm, RepetitionForm
from .models import Band, Repetition, User


def band_form(request):
    if request.method == 'POST':
        user = User()
        band = Band()
        rep = Repetition()
        _band_form = BandForm(request.POST)
        _repetition_form = RepetitionForm(request.POST)
        if _band_form.is_valid() and _repetition_form.is_valid():
            data = _band_form.cleaned_data
            data_rep = _repetition_form.cleaned_data
            band_name = data['band_name']
            band.band_name = band_name
            password = data['password']
            idi = user_reg(band_name, password, user)
            band.user = idi
            band.phone = data['phone']
            band.save()
            rep.band = band
            rep.start_time = data_rep['start_time']
            rep.end_time = data_rep['end_time']
            rep.date = data_rep['date']
            rep.save()
            return redirect('/')
        else:
            print("Форма заполнена не корректно")
    else:
        _band_form = BandForm()
        _repetition_form = RepetitionForm()
    return render(request, 'auth.html', {'band_form': _band_form, 'rep_form': _repetition_form})


def user_reg(username, password, user):
    user.username = username
    user.set_password(password)
    user.save()
    return user