Как исправить обработчик форм, чтобы он работал?

Question

[AprilSnow]

Не получается реализовать форму постинга записи рядом с теми записями, которые уже есть.

Форма:

class CinemaForm(forms.Form):
    th_name = forms.CharField(label='Название кинотеатра')
    time = forms.DateTimeField(label='Сеансы')
    address = forms.CharField(label='Адрес кинотеатра')
    cn_name = forms.CharField(label='Название фильма')

Модель:

class Cinema(models.Model):
    theater_name = models.TextField(max_length = 35, blank=False)
    theater_address = models.TextField(max_length=35, blank=False)
    cinema_timetable = models.DateTimeField(blank=False)
    cinema_name = models.TextField(max_length=35, blank=False)

Представление:

def cinema(request):
    all_cinema = Cinema.objects.all()
    cinema_form = CinemaForm
    if request.POST:
        newpost_form = CinemaForm(request.POST)
        if newpost_form.is_valid():
            newpost_form.save()
            return redirect("/")
    return render_to_response("theatername.html", {"cinema": all_cinema, "form": cinema_form})

Шаблон:

{% block theater %}
{% for theater in cinema %}
<table class="table table-bordered">
<thead>
    <tr>
      <th>Адрес</th>
      <th>Название кинотеатра</th>
    </tr>
  </thead>
<tbody>
    <tr>
        <td>{{ theater.theater_address }}</td>
        <td>{{ theater.theater_name }}</td>
    </tr>
  </tbody>
</table>
<br>
{% endfor %}
<form action="/" method="post">
{% csrf_token %}
{{ form.as_ul }}
<input class="btn btn-success" type="submit" value="Добавить">
{% endblock %}

{% block seance %}
{% for theater in cinema %}
<table class="table table-bordered">
  <thead>
    <tr>
      <th>Название кинотеатра</th>
      <th>Расписание</th>
    </tr>
  </thead>
  <tbody>
    <tr>
        <td>{{ theater.theater_name }}</td>
        <td>{{ theater.cinema_timetable }} - {{ theater.cinema_name }}</td>
    </tr>
  </tbody>
</table>
<br>
{% endfor %}
<form action="/" method="post">
{{ form.as_ul }}
<input class="btn btn-success" type="submit" value="Добавить">
{% endblock %}

Таблицы в бд не создаются, что нужно исправить?

Answer 1

[Roman K]

Нет такого метода (save) у формы. Вам нужно использовать ModelForm.

class CinemaForm(forms.ModelForm):
    class Meta:
        model = Cinema
        fields = ('theater_name', 'theater_address', 'cinema_timetable', 'cinema_name')

Comments

[sim3x]

https://github.com/django/django/blob/master/djang...

[Roman K]

sim3x: Пруф моих слов? :)

[sim3x]

Roman Kitaev: True

[AprilSnow]

Roman Kitaev: Вы не могли бы подсказать, что делать дальше? Джанго все равно выбрасывает проблему с scrf токеном

[AprilSnow]

AprilSnow: csrf*

[Roman K]

AprilSnow: В вашем шаблоне два открывающихся тега с методом POST и только один csft token. При этом ни одного закрывающегося.

[AprilSnow]

Roman Kitaev:

{% csrf_token %}
{{ form.as_ul }}


Не помогло, что может быть не так?

[AprilSnow]

AprilSnow: html отклеился. В общем форму закрыл

[Roman K]

AprilSnow:
1) Зачем вам две формы?
2) cinema_form = CinemaForm - Так вы присваиваете класс, а не экземпляр. Надо скобочки не забывать.

[AprilSnow]

Roman Kitaev: Нужно сделать две разные таблицы, которые буду независимы друг от друга. Думал потом доделать некоторые части, такие как раздельные поля

[AprilSnow]

AprilSnow: Если убрать вторую форму, то джанго все равно выдает ошибку

[Roman K]

AprilSnow: И каким это магическим образом вы хотите сделать две независимые таблицы через один класс формы?

[AprilSnow]

Roman Kitaev: Эм, я хотел позже сделать два класса

[Roman K]

AprilSnow: В общем, думать над архитектурой приложения - дело ваше. Форму советую инициализировать в общем блоке, а не if request.POST. Ну и любая форма должна выглядеть в шаблоне примерно так:

<form method="post">{% csrf_token %}
    {{ form.as_p }}
    <input type="submit"/>
</form>

Как вы могли заметить, здесь нет action-параметра. Советую его указывать явно только в случае, когда это действительно необходимо.

[AprilSnow]

Roman Kitaev: Все равно вылетает ошибка csrf токена, попробовал убрать второй блок из шаблона - не помогло

[Roman K]

AprilSnow: Значит, вы где-то ошиблись. Код вьюхи и целого шаблона в студию.

[AprilSnow]

Roman Kitaev: https://gist.github.com/aprilsnow7870/cb4087b1ead7...

[Roman K]

AprilSnow:
Вьюха:

def cinema(request):
    form = CinemaForm(request.POST)
    	if request.POST and form.is_valid():
			        newpost_form.save()
        			return redirect("/")
    	return render_to_response("theatername.html", {"cinema": Cinema.objects.all(), "form": form})

Одна вьюха не может обработать две формы сразу (откуда ей знать, из какой именно вы ей чего послали). Поэтому, на вторую форму придётся ставить action и писать вторую вьюху.

[Roman K]

Roman Kitaev: Код не тестировался. Если появится ошибка, поставьте form = CinemaForm(request.POST or {})

[AprilSnow]

Roman Kitaev: И все равно джанго выкидывает ошибку csrf токена

[Roman K]

AprilSnow: А вы точно разрабатываете не на продакшен-сервере и перезагружали его после изменений?

[AprilSnow]

Roman Kitaev: Точно, на скриншоте видно, что локалка

[AprilSnow]

Roman Kitaev: Может быть, скину весь проект на гит?

[AprilSnow]

AprilSnow: гитхаб*

[Roman K]

AprilSnow: Кажется, я вспомнил. Используйте вместе render_to_response просто render из django.shortcuts. Использовать его так же, как и render_to_response, только первым аргументом ставьте реквест. Например, так: return render(request, "theatername.html", {"cinema": Cinema.objects.all(), "form": form})

[AprilSnow]

Roman Kitaev: Почти помогло, ошибки csrf токена теперь нет, но новый пост не добавляется

[Roman K]

AprilSnow: В таком случае, попробуйте использовать CBV (в конечном итоге, вы всё равно к ним придёте. У меня во всём проекте только одна вьюха не CBV).

from django.views.generic import *

class CinemaCreate(CreateView):
    form_class = CinemaForm
    template_name = 'theatername.html'

    def get_context_data(self, **kwargs)
        c = super().get_context_data(**kwargs)
        c['cinema'] = Cinema.objects.all()  # Это чтобы в контекст шли ваши Cinem'ы
        return c

[AprilSnow]

Roman Kitaev: Теперь проблемы с __init__. Он ожидает один аргумент, а принимает два.

[AprilSnow]

Roman Kitaev: Разве form_class и template_name не нужно было объявить под юнитом и передать в метод?

[Roman K]

AprilSnow: А вы в туалет до сих пор с родителями ходите, если не секрет?