Django - CreateView не сохраняет форму с вложенным набором форм

14

Я пытаюсь адаптировать подход для сохранения вложенных наборов форм с основной формой с помощью функции макета Django-Crispy-Forms, но не могу сохранить ее. Я слежу за этим примером кода, но не смог проверить набор форм для сохранения данных. Я буду очень благодарен, если кто-то может указать на мою ошибку. Мне также нужно добавить три встроенных в том же виде для EmployeeForm. Я попробовал Django-Extra-Views, но не смог сделать это. Буду признателен, если вы посоветуете добавить более одной строчки для одного и того же представления, например, около 5. Все, чего я хочу, это создать единую страницу для создания Employeeи ее строчки, как Education, Experience, Others. Ниже приведен код:

модели:

class Employee(models.Model):
    user = models.OneToOneField(User, on_delete=models.CASCADE, related_name='employees',
                                null=True, blank=True)
    about = models.TextField()
    street = models.CharField(max_length=200)
    city = models.CharField(max_length=200)
    country = models.CharField(max_length=200)
    cell_phone = models.PositiveIntegerField()
    landline = models.PositiveIntegerField()

    def __str__(self):
        return '{} {}'.format(self.id, self.user)

    def get_absolute_url(self):
        return reverse('bars:create', kwargs={'pk':self.pk})

class Education(models.Model):
    employee = models.ForeignKey('Employee', on_delete=models.CASCADE, related_name='education')
    course_title = models.CharField(max_length=100, null=True, blank=True)
    institute_name = models.CharField(max_length=200, null=True, blank=True)
    start_year = models.DateTimeField(null=True, blank=True)
    end_year = models.DateTimeField(null=True, blank=True)

    def __str__(self):
        return '{} {}'.format(self.employee, self.course_title)

Посмотреть:

class EmployeeCreateView(CreateView):
    model = Employee
    template_name = 'bars/crt.html'
    form_class = EmployeeForm
    success_url = None

    def get_context_data(self, **kwargs):
        data = super(EmployeeCreateView, self).get_context_data(**kwargs)
        if self.request.POST:
            data['education'] = EducationFormset(self.request.POST)
        else:
            data['education'] = EducationFormset()
        print('This is context data {}'.format(data))
        return data


    def form_valid(self, form):
        context = self.get_context_data()
        education = context['education']
        print('This is Education {}'.format(education))
        with transaction.atomic():
            form.instance.employee.user = self.request.user
            self.object = form.save()
            if education.is_valid():
                education.save(commit=False)
                education.instance = self.object
                education.save()

        return super(EmployeeCreateView, self).form_valid(form)

    def get_success_url(self):
        return reverse_lazy('bars:detail', kwargs={'pk':self.object.pk})

Формы:

class EducationForm(forms.ModelForm):
    class Meta:
        model = Education
        exclude = ()
EducationFormset =inlineformset_factory(
    Employee, Education, form=EducationForm,
    fields=['course_title', 'institute_name'], extra=1,can_delete=True
    )

class EmployeeForm(forms.ModelForm):

    class Meta:
        model = Employee
        exclude = ('user', 'role')

    def __init__(self, *args, **kwargs):
        super(EmployeeForm, self).__init__(*args, **kwargs)
        self.helper = FormHelper()
        self.helper.form_tag = True
        self.helper.form_class = 'form-horizontal'
        self.helper.label_class = 'col-md-3 create-label'
        self.helper.field_class = 'col-md-9'
        self.helper.layout = Layout(
            Div(
                Field('about'),
                Field('street'),
                Field('city'),
                Field('cell_phone'),
                Field('landline'),
                Fieldset('Add Education',
                    Formset('education')),
                HTML("<br>"),
                ButtonHolder(Submit('submit', 'save')),
                )
            )

Пользовательский объект макета в соответствии с примером:

from crispy_forms.layout import LayoutObject, TEMPLATE_PACK
from django.shortcuts import render
from django.template.loader import render_to_string

class Formset(LayoutObject):
    template = "bars/formset.html"

    def __init__(self, formset_name_in_context, template=None):
        self.formset_name_in_context = formset_name_in_context
        self.fields = []
        if template:
            self.template = template

    def render(self, form, form_style, context, template_pack=TEMPLATE_PACK):
        formset = context[self.formset_name_in_context]
        return render_to_string(self.template, {'formset': formset})

Formset.html:

{% load static %}
{% load crispy_forms_tags %}
{% load staticfiles %}

<table>
{{ formset.management_form|crispy }}

    {% for form in formset.forms %}
            <tr class="{% cycle 'row1' 'row2' %} formset_row-{{ formset.prefix }}">
                {% for field in form.visible_fields %}
                <td>
                    {# Include the hidden fields in the form #}
                    {% if forloop.first %}
                        {% for hidden in form.hidden_fields %}
                            {{ hidden }}
                        {% endfor %}
                    {% endif %}
                    {{ field.errors.as_ul }}
                    {{ field|as_crispy_field }}
                </td>
                {% endfor %}
            </tr>
    {% endfor %}

</table>
<br>
<script src="//ajax.googleapis.com/ajax/libs/jquery/2.1.3/jquery.min.js">
</script>
<script src="{% static 'js/jquery.formset.js' %}">
</script>
<script type="text/javascript">
    $('.formset_row-{{ formset.prefix }}').formset({
        addText: 'add another',
        deleteText: 'remove',
        prefix: '{{ formset.prefix }}',
    });
</script>

Там нет ошибок в терминале и / или иным образом. Помощь очень ценится.

Шазия Нусрат
источник
Альтернативное решение состоит в том, чтобы форма также обрабатывала набор форм: я делаю это, используя cached_property для связанного набора форм в schinckel.net/2019/05/23/form-and-formset
Мэтью Шинкель,

Ответы:

0

В настоящее время вы не правильно обрабатываете форму CreateView. form_validв этом представлении будет обрабатываться только родительская форма, а не наборы форм. Что вы должны сделать, это переопределить postметод, и там вам нужно проверить как форму, так и любые формы, которые к ней прикреплены:

def post(self, request, *args, **kwargs):
    form = self.get_form()
    # Add as many formsets here as you want
    education_formset = EducationFormset(request.POST)
    # Now validate both the form and any formsets
    if form.is_valid() and education_formset.is_valid():
        # Note - we are passing the education_formset to form_valid. If you had more formsets
        # you would pass these as well.
        return self.form_valid(form, education_formset)
    else:
        return self.form_invalid(form)

Затем вы модифицируете form_validтак:

def form_valid(self, form, education_formset):
    with transaction.atomic():
        form.instance.employee.user = self.request.user
        self.object = form.save()
        # Now we process the education formset
        educations = education_formset.save(commit=False)
        for education in educations:
            education.instance = self.object
            education.save()
        # If you had more formsets, you would accept additional arguments and
        # process them as with the one above.
    # Don't call the super() method here - you will end up saving the form twice. Instead handle the redirect yourself.
    return HttpResponseRedirect(self.get_success_url())

То, как вы сейчас используете, get_context_data()неверно - полностью удалите этот метод. Он должен использоваться только для извлечения данных контекста для визуализации шаблона. Вы не должны вызывать это из вашего form_valid()метода. Вместо этого вам нужно передать formset этому методу из post()метода, как описано выше.

Я оставил несколько дополнительных комментариев в приведенном выше примере кода, которые, надеюсь, помогут вам разобраться в этом.

solarissmoke
источник
Пожалуйста, заново создайте пример, прежде чем ответить. Я попробовал ваш кусок, но не работает.
Шазия Нусрат
1
@ShaziaNusrat извините, у меня нет времени, чтобы попытаться выяснить, что не работает для вас, особенно если вы не говорите, что пытались, а что не работали («Это не работает» не адекватное описание того, что не сработало). Я считаю, что в моем ответе достаточно, чтобы помочь вам определить, что вам нужно изменить в текущей реализации. Если нет, то будем надеяться, что кто-то еще сможет дать вам более полный ответ.
Solarissmoke
Я попробовал это в коде для тестирования, и он работал с проблемами. Вот почему я смиренно прошу вас попробовать это на вашей стороне, чтобы вы могли лучше вести меня. Я благодарен, что вы нашли время, чтобы помочь мне. Но не работает.
Шазия Нусрат
0

Может быть, вы хотели бы видеть пакет django-extra-views, который обеспечивает представление CreateWithInlinesView, ведь позволяет вам создавать формы с вложенными строками, такими как строки Django-admin.

В вашем случае это было бы что-то вроде этого:

views.py

class EducationInline(InlineFormSetFactory):
    model = Education
    fields = ['course_title', 'institute_name']


class EmployeeCreateView(CreateWithInlinesView):
    model = Employee
    inlines = [EducationInline,]
    fields = ['about', 'street', 'city', 'cell_phone', 'landline']
    template_name = 'bars/crt.html'

crt.html

<form method="post">
  ...
  {{ form }}
  <table>
  {% for formset in inlines %}
    {{ formset.management_form }}
      {% for inline_form in formset %}
        <tr class="{% cycle 'row1' 'row2' %} formset_row-{{ formset.prefix }}">
          {{ inline_form }}
        </tr>
      {% endfor %}
  {% endfor %}
  </table>
  ...
  <input type="submit" value="Submit" />
</form>

<script src="//ajax.googleapis.com/ajax/libs/jquery/2.1.3/jquery.min.js">
</script>
<script src="{% static 'js/jquery.formset.js' %}">
</script>
<script type="text/javascript">
    {% for formset in inlines %}
      $('.formset_row-{{ formset.prefix }}').formset({
          addText: 'add another',
          deleteText: 'remove',
          prefix: '{{ formset.prefix }}',
      });
    {% endfor %}
</script>

Представление EmployeeCreateViewбудет обрабатывать формы для вас, как в Django-admin. С этого момента вы можете применить желаемый стиль к формам.

Я рекомендую вам посетить документацию для получения дополнительной информации

Отредактировано: я добавил management_form и кнопки JS, чтобы добавить / удалить.

Джон
источник
Я уже пробовал это, но это не позволит мне добавить / удалить кнопки для нескольких строк. Он поддерживает только одну строку с кнопками JS. Я уже пробовал это.
Шазия Нусрат
1
Это поддерживает, вы должны добавить management_formдля каждогоformset
Джон
0

Вы сказали, что есть ошибка, но вы не показываете ее в своем вопросе. Ошибка (и вся трассировка) более важна, чем все, что вы написали (кроме тех, которые могут быть из форм .py и views.py).

Ваш случай немного сложнее из-за наборов форм и использования нескольких форм в одном CreateView. В интернете не так много (или не так много хороших) примеров. Пока вы не покопаетесь в коде django о том, как работают встроенные формы, у вас будут проблемы.

Хорошо, прямо к делу. Ваша проблема в том, что формы не инициализируются с тем же экземпляром, что и ваша основная форма. И когда ваша аминная форма сохраняет данные в базе данных, экземпляр в наборе форм не изменяется, и в конце у вас нет идентификатора основного объекта, чтобы его можно было использовать в качестве внешнего ключа. Изменение атрибута экземпляра атрибута формы после init не является хорошей идеей.

В обычных формах, если вы измените его после is_valid, вы получите непредсказуемые результаты. Для наборов форм изменение атрибута экземпляра даже непосредственно после init ничего не даст, поскольку формы в наборе форм уже инициализированы с некоторым экземпляром, и изменение его после не поможет. Хорошей новостью является то, что вы можете изменить атрибуты экземпляра после инициализации Formset, потому что все атрибуты экземпляра форм будут указывать на один и тот же объект после инициализации Formset.

У вас есть два варианта:

Вместо установки атрибута экземпляра, если используется formset, установите только instance.pk. (Это просто предположение, я никогда не делал это, но я думаю, что это должно работать. Проблема в том, что это будет выглядеть как хак). Создайте форму, которая будет инициализировать все формы / наборы форм одновременно. Когда вызывается метод is_valid (), все fomrs должны быть проверены. При вызове метода save () все формы должны быть сохранены. Затем вам нужно установить атрибут form_class вашего CreateView для этого класса формы. Единственная сложность заключается в том, что после инициализации вашей основной формы вам нужно инициализировать другие (formsests) экземпляром вашей первой формы. Также вам нужно установить формы / наборы форм как атрибуты вашей формы, чтобы иметь доступ к ним в шаблоне. Я использую второй подход, когда мне нужно создать объект со всеми связанными объектами.

инициализированные некоторыми данными (в данном случае данными POST), проверенными на достоверность с помощью is_valid (), могут быть сохранены с помощью save (), когда они действительны. Вы сохраняете интерфейс формы, и если вы сделали свою форму правильно, вы даже можете использовать ее не только для создания, но и для обновления объектов вместе со связанными объектами, и представления будут очень простыми.

Алексис Руксель
источник