Django - CreateView ไม่บันทึกฟอร์มด้วยชุดรูปแบบซ้อน


14

ฉันกำลังพยายามปรับวิธีการบันทึกชุดรูปแบบที่ซ้อนอยู่ด้วยแบบฟอร์มหลักโดยใช้คุณสมบัติการจัดวางแบบ Django-Crispy-Forms แต่ฉันไม่สามารถบันทึกได้ ฉันกำลังดังต่อไปนี้โครงการตัวอย่างรหัส แต่อาจไม่ได้รับการตรวจสอบ formset ในการบันทึกข้อมูล ฉันจะขอบคุณจริงๆถ้ามีคนชี้ให้เห็นความผิดพลาดของฉัน ฉันต้องเพิ่ม inline สามรายการในมุมมองเดียวกันสำหรับ EmployeeForm ฉันลองใช้ Django-Extra-Views แต่ไม่สามารถทำงานได้ จะขอบคุณถ้าคุณให้คำแนะนำสำหรับการเพิ่มมากกว่าหนึ่ง inlines สำหรับมุมมองเช่นเดียวกับรอบ 5. ทั้งหมดที่ฉันต้องการที่จะประสบความสำเร็จที่หน้าเดียวสำหรับการสร้างEmployeeและ inlines 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')),
                )
            )

Custom Layout Object ตามตัวอย่าง:

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>

ไม่มีข้อผิดพลาดใน terminal และหรือเป็นอย่างอื่น ช่วยชื่นชมมาก


ทางเลือกอื่นคือให้ form จัดการกับ formset ด้วย: ฉันใช้ cached_property สำหรับ formset ที่เกี่ยวข้องในschinckel.net/2019/05/23/form-and-formset
Matthew Schinckel

คำตอบ:


0

คุณยังไม่ได้ประมวลผล formset 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()วิธีที่อธิบายไว้ข้างต้น

ฉันได้แสดงความคิดเห็นเพิ่มเติมในโค้ดตัวอย่างด้านบนซึ่งหวังว่าจะช่วยให้คุณเข้าใจได้


โปรดสร้างตัวอย่างอีกครั้งในพื้นที่ก่อนที่จะตอบ ฉันลองชิ้นส่วนของคุณแล้ว แต่ไม่ทำงาน
Shazia Nusrat

1
@ShaziaNusrat ขออภัยฉันไม่มีเวลาลองและทำงานในสิ่งที่ไม่ได้ผลสำหรับคุณโดยเฉพาะถ้าคุณไม่พูดในสิ่งที่คุณได้ลองและสิ่งที่ไม่ได้ผล ("มันใช้งานไม่ได้" ไม่ใช่ คำอธิบายที่เพียงพอของสิ่งที่ใช้ไม่ได้) ฉันเชื่อว่ามีคำตอบของฉันเพียงพอที่จะช่วยคุณระบุสิ่งที่คุณต้องการเปลี่ยนแปลงกับการใช้งานในปัจจุบันของคุณ ถ้าไม่เช่นนั้นเราหวังว่าคนอื่นจะสามารถให้คำตอบที่ครอบคลุมยิ่งขึ้นแก่คุณ
solarissmoke

ฉันลองในรหัสเพื่อทดสอบและมันก็วิ่งด้วยปัญหา นั่นเป็นสาเหตุที่ฉันขอร้องให้คุณลองทำในด้านของคุณเพื่อให้คุณสามารถแนะนำฉันได้ดีขึ้น ฉันรู้สึกขอบคุณที่คุณใช้เวลาช่วยฉัน แต่ไม่ทำงาน
Shazia Nusrat

0

บางทีคุณอาจต้องการที่จะเห็นแพคเกจdjango-extra-viewsให้มุมมองCreateWithInlinesViewแม่มดช่วยให้คุณสามารถสร้างรูปแบบที่มีอินไลน์ซ้อนกันเช่นอินไลน์ Django ผู้ดูแลระบบ

ในกรณีของคุณมันจะเป็นอย่างนั้น:

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 ฉันได้ลองแล้ว
Shazia Nusrat

1
สนับสนุนคุณต้องเพิ่มmanagement_formสำหรับแต่ละformset
John

0

คุณบอกว่ามีข้อผิดพลาด แต่คุณไม่ได้แสดงไว้ในคำถามของคุณ ข้อผิดพลาด (และการติดตามกลับทั้งหมด) มีความสำคัญมากกว่าสิ่งที่คุณเขียน (ยกเว้นอาจมาจาก forms.py และ views.py)

กรณีของคุณมีเล่ห์เหลี่ยมเล็กน้อยเนื่องจากชุดรูปแบบและใช้หลายรูปแบบใน CreateView เดียวกัน มีตัวอย่างไม่มาก (หรือน้อยมาก) บนอินเทอร์เน็ต จนกว่าคุณจะขุดในโค้ด django ว่ารูปแบบอินไลน์ทำงานอย่างไรคุณจะมีปัญหา

ตกลงตรงประเด็น ปัญหาของคุณคือว่า formets ไม่ได้เริ่มต้นด้วยอินสแตนซ์เดียวกันกับฟอร์มหลักของคุณ และเมื่อรูปแบบอามินของคุณบันทึกข้อมูลลงในฐานข้อมูลอินสแตนซ์ในชุดรูปแบบจะไม่เปลี่ยนแปลงและในตอนท้ายคุณจะไม่มี ID ของวัตถุหลักเพื่อที่จะนำไปเป็นคีย์ต่างประเทศ การเปลี่ยนแอตทริบิวต์ของแอตทริบิวต์แบบฟอร์มหลังจาก init ไม่ใช่ความคิดที่ดี

ในรูปแบบปกติหากคุณเปลี่ยนหลังจาก is_valid คุณจะได้ผลลัพธ์ที่ไม่คาดคิด สำหรับชุดรูปแบบการเปลี่ยนแอตทริบิวต์ของอินสแตนซ์แม้โดยตรงหลังจาก init จะไม่ทำเช่นนั้นเนื่องจากรูปแบบในชุดรูปแบบจะเริ่มต้นได้แล้วด้วยบางอินสแตนซ์และการเปลี่ยนรูปแบบหลังจากจะไม่ช่วย ข่าวดีก็คือคุณสามารถเปลี่ยนคุณสมบัติของอินสแตนซ์หลังจากที่เริ่มต้น Formset ได้เนื่องจากแอตทริบิวต์ของอินสแตนซ์ของฟอร์มทั้งหมดจะชี้ไปที่วัตถุเดียวกันหลังจากที่เริ่มต้นชุดรูปแบบแล้ว

คุณมีสองทางเลือก:

แทนที่จะตั้งค่าแอตทริบิวต์ instance หาก formset ให้ตั้งค่า instance.pk เท่านั้น (นี่เป็นเพียงการคาดเดาที่ฉันไม่เคยทำ แต่ฉันคิดว่ามันควรจะทำงานได้ปัญหาคือมันจะดูเป็นแฮ็ค) สร้างแบบฟอร์มที่จะเริ่มต้นทุกรูปแบบ / ชุดรูปแบบในครั้งเดียว เมื่อ is_valid () วิธีการที่เรียกว่า fomrs ทั้งหมดควรได้รับการตรวจสอบ เมื่อมันเป็นวิธีการบันทึก () เรียกว่าทุกรูปแบบจะต้องบันทึก จากนั้นคุณต้องตั้งค่าแอตทริบิวต์ form_class ของ CreateView ให้เป็นคลาสฟอร์มนั้น ส่วนที่ยุ่งยากเท่านั้นคือหลังจากที่แบบฟอร์มหลักของคุณเริ่มต้นแล้วคุณต้องเริ่มต้นแบบฟอร์มอื่น ๆ (แบบฟอร์ม) ด้วยอินสแตนซ์ของแบบฟอร์มแรกของคุณ นอกจากนี้คุณต้องตั้งค่าฟอร์ม / ชุดรูปแบบเป็นคุณลักษณะของแบบฟอร์มของคุณเพื่อให้สามารถเข้าถึงได้ในแม่แบบ ฉันใช้วิธีที่สองเมื่อฉันต้องการสร้างวัตถุที่มีวัตถุที่เกี่ยวข้องทั้งหมด

เริ่มต้นด้วยข้อมูลบางอย่าง (ในกรณีนี้ข้อมูล POST) ตรวจสอบความถูกต้องด้วย is_valid () สามารถบันทึกด้วย save () เมื่อถูกต้อง คุณรักษาอินเทอร์เฟซของแบบฟอร์มและถ้าคุณทำแบบฟอร์มของคุณอย่างถูกต้องคุณสามารถใช้มันไม่เพียง แต่สำหรับการสร้าง แต่สำหรับการปรับปรุงวัตถุร่วมกับวัตถุที่เกี่ยวข้องและมุมมองจะง่ายมาก

โดยการใช้ไซต์ของเรา หมายความว่าคุณได้อ่านและทำความเข้าใจนโยบายคุกกี้และนโยบายความเป็นส่วนตัวของเราแล้ว
Licensed under cc by-sa 3.0 with attribution required.