python - 自定义 Django Crispy 表单中忽略了 ValidationError

标签 python django forms validation django-crispy-forms

我有一个相当复杂的 Django 表单,它影响 3 个模型,其中一部分包括一个内联表单集。我在 https://dev.to/zxenia/django-inline-formsets-with-class-based-views-and-crispy-forms-14o6 找到了构建表单的一个很好的解决方案。 。我扩展了该解决方案,并以与添加表单集类似的方式添加了第三个模型(使用自定义 Django Crispy Form 并使用 Crispy Forms 布局功能插入它)。

我的问题是,两个插入表单(表单集和小子表单)中出现的任何验证错误都会被简单地忽略 - 主表单正确发布并引发的 ValidationErrors 在表单中显示为允许用户更正的错误任何错误及其数据都会正确保存到数据库中。如果子表单和表单集有效,它们的数据也会正确保存。但是,如果子表单和表单集中的数据无效,则表单永远不会显示错误以使用户有机会纠正错误,并且数据会被简单地忽略并且永远不会保存到数据库中 - 主模型的数据保存得很好不过。

我的问题是,如何刷新表单并在添加的子表单和表单集中显示错误,从而允许用户纠正错误?

下面的大部分代码来自上面引用的相当好的帖子,并添加了第三个模型

型号:

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


class Collection(models.Model):
    subject = models.CharField(max_length=300, blank=True)
    owner = models.CharField(max_length=300, blank=True)
    note = models.TextField(blank=True)
    created_by = models.ForeignKey(User,
        related_name="collections", blank=True, null=True,
        on_delete=models.SET_NULL)

    def __str__(self):
        return str(self.id)


class CollectionTitle(models.Model):
    """
    A Class for Collection titles.

    """
    collection = models.ForeignKey(Collection,
        related_name="has_titles", on_delete=models.CASCADE)
    name = models.CharField(max_length=500, verbose_name="Title")
    language = models.CharField(max_length=3)

Class CollectionTxn(models.Model):
    """
    A Class for Collection transactions.

    """
    collection = models.ForeignKey(Collection,
        related_name="has_txn", on_delete=models.CASCADE)
    number_received= models.IntegerField()
    date_received= models.DateField()

    class Meta:
        '''
        If 2 rows are entered with the same information, a validation error is raised, but it just
        doesn't save the data at all instead of refreshing the form showing the error.
        '''
        unique_together = ('number_received', 'date_received')

表单.py:

from django import forms
from .models import Collection, CollectionTitle
from django.forms.models import inlineformset_factory
from crispy_forms.helper import FormHelper
from crispy_forms.layout import Layout, Field, Fieldset, Div, Row, HTML, ButtonHolder, Submit
from .custom_layout_object import Formset, Subform

import re


class CollectionTitleForm(forms.ModelForm):

    class Meta:
        model = CollectionTitle
        exclude = ()

    def __init__(self, *args, **kwargs):
        super().__init__(*args, **kwargs)

        formtag_prefix = re.sub('-[0-9]+$', '', kwargs.get('prefix', ''))

        self.helper = FormHelper()
        self.helper.form_tag = False
        self.helper.layout = Layout(
            Row(
                Field('name'),
                Field('language'),
                Field('DELETE'),
                css_class='formset_row-{}'.format(formtag_prefix)
            )
        )


CollectionTitleFormSet = inlineformset_factory(
    Collection, CollectionTitle, form=CollectionTitleForm,
    fields=['name', 'language'], extra=1, can_delete=True
)


class CollectionForm(forms.ModelForm):

    class Meta:
        model = Collection
        exclude = ['created_by', ]

    def __init__(self, *args, **kwargs):
        super(CollectionForm, 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('subject'),
                Field('owner'),
                Fieldset('Add titles',
                         Formset('titles')),
                Field('note'),
                Subform('transactions'),
                HTML("<br>"),
                ButtonHolder(Submit('submit', 'Save')),
            )
        )

class CollectionTxnForm(forms.ModelForm):
    
    class Meta:
        model = CollectionTxn
        exclude = ()

    def __init__(self, *args, **kwargs):
        super().__init__(*args, **kwargs)
        self.fields['collection'].widget = HiddenInput()
        self.helper = FormHelper()
        self.helper.form_tag = False
        self.helper.layout = Layout(
            Row(
                Field('number_received'),
                Field('date_received'),
            )
        )

views.py:

from .models import *
from .forms import *
from django.views.generic.edit import CreateView, UpdateView
from django.urls import reverse_lazy
from django.db import transaction

class CollectionCreate(CreateView):
    model = Collection
    template_name = 'mycollections/collection_create.html'
    form_class = CollectionForm
    success_url = None

    def get_context_data(self, **kwargs):
        data = super(CollectionCreate, self).get_context_data(**kwargs)
        if self.request.POST:
            data['titles'] = CollectionTitleFormSet(self.request.POST)
            data['transactions'] = CollectionTrxForm(self.request.POST)
        else:
            data['titles'] = CollectionTitleFormSet()
            data['transactions'] = CollectionTrxForm()
        return data

    def form_valid(self, form):
        context = self.get_context_data()
        titles = context['titles']
        transactions = context['transactions']
        with transaction.atomic():
            form.instance.created_by = self.request.user
            self.object = form.save()
            if titles.is_valid():
                titles.instance = self.object
                titles.save()
            if transactions.is_valid():
                transactions.save()
        return super(CollectionCreate, self).form_valid(form)

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

自定义布局对象.py

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 = "mycollections/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})

class SubForm(LayoutObject):
    template = "mycollections/subform.html"

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

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

表单集.html

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

<style type="text/css">
  .delete-row {
    align-self: center;
  }
</style>

{{ formset.management_form|crispy }}

{% for form in formset.forms %}
  {% for hidden in form.hidden_fields %}
      {{ hidden|as_crispy_field }}
  {% endfor %}
  {% crispy form %}
{% endfor %}

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

子表单.html

{% load crispy_forms_tags %}
{% crispy subform %}

collection_create.html

{% extends "mycollections/base.html" %}
{% load crispy_forms_tags %}
{% block content %}
<div class="container">
    <div class="card">
        <div class="card-header">
            Create collection
        </div>
        <div class="card-body">
             {% crispy form %}
        </div>
    </div>
</div>
{% endblock content %}

基本上,对于与添加到布局中的表单集和子表单关联的字段,仍然会引发验证错误,但它们不会向上冒泡到表单级别以显示错误,它们只是被忽略,并且永远不会保存数据。 “主”模型工作正常,并且其字段显示验证错误。如果没有无效数据,则主窗体、子窗体和窗体集的数据都正确保存。如果表单集或子表单中存在无效数据,用户将永远没有机会更正该数据。

如果我在哪里添加所需的代码,以便在表单集或子表单中输入任何无效数据会导致表单刷新显示错误,而不是仅仅忽略和不保存无效数据,请提供任何帮助。

最佳答案

经过多次调试和分析我自己的代码和 Crispy 的代码,我已经解决了我自己的问题。只需检查子表单和表单集验证,如果无效,则重新呈现表单。

这是 View 中执行此操作的新 form_valid() 方法:

    def form_valid(self, form):
        context = self.get_context_data()
        titles = context['titles']
        transactions = context['transactions']
        with transaction.atomic():
            form.instance.created_by = self.request.user
            if titles.is_valid() and transactions_is_valid():
                self.object = form.save() #only save form if other subforms validate
                titles.instance = self.object
                # Any other field processing goes here
                titles.save()
                transactions.save()
            else:
                # If any subform or subformset is invalid, re-render the form showing errors
                context.update({'titles': titles})
                context.update({'transactions': transactions})
                return self.render_to_response(context)
        return super(CollectionCreate, self).form_valid(form)

有了这个,表单集和子表单中的任何干净方法或者如果存在任何其他错误(例如,如果表单集没有两行可以相同,因为在模型中声明了 unique_together()),表单将刷新显示所有所有三个组合表单/表单集上的错误允许用户更正这些错误。

现在 - 使用 Ajax 来完成,这样页面就不会刷新:)

关于python - 自定义 Django Crispy 表单中忽略了 ValidationError,我们在Stack Overflow上找到一个类似的问题: https://stackoverflow.com/questions/62861445/

相关文章:

python - Selenium 在单击后给出服务器错误 500(Django、pytest)

javascript - 使用书签更改文本框大小

php - Symfony2 : imbricate a formType in another formType refer to the foreign key for addAction() method (multiple form Type)

python - 有没有办法在 BigQuery 中对时间序列数据重新采样?

python - 无法导入安装有 `pip install -e .` 的本地包

python - BeautifulSoup 理解相对 URL 吗?

python - 如何使用 PySpark 计算 ADLS 中的目录大小?

python - @staff_member_required 正在抛出 "object has no attribute ' 用户'”

Django Rest 'api-docs' 不是注册的命名空间

javascript - 为什么 <form> 上的 onChange 事件处理程序从 <select> 获取分派(dispatch)的事件,而不是从 &lt;input&gt; 获取?