Как сохранить запись?

Question

[Александр]

Доброй ночи, просидел 2 часа над вопросом как сохранить запись в бд? Вот код

#form
class AddPerformerForm(ModelForm):
    categories = forms.ModelChoiceField(queryset=Category.objects.all(),
                                        widget=forms.CheckboxSelectMultiple,
                                        empty_label=None)

    class Meta:
        model = Performer
        fields = ['user', 'categories']
#models
class Category(models.Model):
    name = models.CharField(max_length=64)

    def __str__(self):
        return self.name


class Performer(models.Model):
    user = models.ForeignKey(
        settings.AUTH_USER_MODEL,
        on_delete=models.CASCADE,
    )
    categories = models.ManyToManyField(Category)
#view
def create_performer(request):
    form = AddPerformerForm({'user': request.user})
    if request.method == 'POST':
        if form.is_valid():
            form.save()
            return HttpResponseRedirect('/')
        else:
            return HttpResponse('No')
    context = {"form": form}
    return render(request, "performerslist/addperformer.html", context)

<form action="/create-performer/" method="POST">
			{% csrf_token %}
				<div class="row">
				{% for category in form.categories %}						
						{{category.tag}}
						<label for="{{category.id_for_label}}">{{category.choice_label}}</label>
						<br>
				{% endfor %}
				<button class="btn">button</button>
			</div>
		</form>

Дело в том, что форма не проходит валидацию, потому что в форму передается только категории без самого юзера.

Comments

[Степан Крапивин]

А где в шаблоне выводится поле user?

[Александр]

Pavel Denisov, Посмотрите пожалуйста, я исправил, но все равно не получается

class AddPerformerForm(ModelForm):
    categories = forms.ModelChoiceField(queryset=Category.objects.all(),
                                        widget=forms.CheckboxSelectMultiple,
                                        empty_label=None)

    class Meta:
        model = Performer
        exclude = ['user']
        fields = ['categories']

def create_performer(request):
    form = AddPerformerForm(request.POST or None)
    if request.method == 'POST':
        if form.is_valid():
            instance = form.save(commit=False)
            instance.user = request.user
            instance.save()
            return HttpResponseRedirect('/')
        else:
            return HttpResponse('no')
    context = {"form": form}
    return render(request, "performerslist/addperformer.html", context)

[Александр]

Степан, Мне нужно, чтобы user привязывался автоматически, под которым пользователь зашел на сайт

[Степан Крапивин]

AlexMine, это понятно. Но вы пихаете значение в форму, а это значение должно подставится в инпут на странице, но этого инпута нет на странице, а значит и данные о пользователе со страницы не придут. Поэтому форма не проходит валидацию.

[Степан Крапивин]

AlexMine, как вариант, выводить на странице этот инпут скрытым.

[Александр]

Добавил User в форму и сделал его не обязательным
Степан,

class AddPerformerForm(ModelForm):
    categories = forms.ModelChoiceField(queryset=Category.objects.all(),
                                        widget=forms.CheckboxSelectMultiple,
                                        empty_label=None)
    user = forms.CharField(required=False)

    class Meta:
        model = Performer
        fields = ['user', 'categories']

def create_performer(request):
    if request.method == 'POST':
        form = AddPerformerForm(request.POST or None)
        if form.is_valid():
            instance = form.save(commit=False)
            instance.user = request.user
            instance.save()
            return HttpResponseRedirect('/')
        else:
            return HttpResponse(request.POST)
    else:
        form = AddPerformerForm(request.POST or None)
        context = {"form": form}
        return render(request, "performerslist/addperformer.html", context)

Теперь мне Django говорит Cannot assign "''": "Performer.user" must be a "User" instance.

Как сделать, чтобы request.user был экземпляром класса?

[Степан Крапивин]

AlexMine, а что мешало тупо добавить скрытый инпут в форму, как я писал ранее?

[Александр]

Степан, Дело в том, что я добавил, но теперь django жалуется, что я передаю в поле user просто строку, а надо экземпляр класса

[Степан Крапивин]

Pavel Denisov, ну да, доступен, но это самое простое решение, которое уже сразу заработает. Человек начал заморачиваться с form.is_valid() и save(commit=False) и запутался.

[Александр]

Pavel Denisov, Я переопределяю в шаблоне, потому что иначе materializecss не показывает кнопки.
А так спасибо за наводки в комментариях выше с помощью них разобрался с этим вопросом.

[Александр]

Степан, Спасибо за комментарии, в вопросе разобрался.

Answer 1

[Дмитрий Матвеев]

form = AddPerformerForm({'user': request.user})
В этой строчке вы инициализируете форму только с юзером, не передавая обязательные categories. Конечно она не пройдёт валидацию.

if request.method == 'POST':
    form = AddPerformerForm({'user': request.user, 'categories': request.POST['ключ-с-категориями']})
    if form.is_valid():
        form.save()
        return HttpResponseRedirect('/')
    else:
        return HttpResponse('No')
form = AddPerformerForm()
context = {"form": form}
return render(request, "performerslist/addperformer.html", context)