python - django 1.5 ModelForm 中的 "This field cannot be null"错误

Question

我设法创建了一个 ModelForm，它基本上在链接到环境上的应用程序的数据库中插入一个警报对象，该对象将向联系人发送邮件(使用外键管理)。我的表单(基于 CreateView 基于类的表单)由 3 个字段组成:

• 应用程序的 CharField
• 环境的 ModelChoiceField
• 联系人的电子邮件字段

CreateView 与一些 AJAX 逻辑混合在一起，使提交内容整体动态化(即绘制错误工具提示，而无需重新加载每个字段旁边的页面，而不是在表单上方出现大错误)。

ModelForm 的逻辑是:

1. 检查数据库中是否已存在该应用程序。否则会引发 ValidationError
2. 检查联系邮件地址是否存在于数据库中。否则它会创建它
3. 检查要创建的结果警报对象是否已存在于数据库中。如果存在，则会引发 ValidationError。

到目前为止一切正常，除了当我想使用不存在的联系邮件地址提交表单时，ModelForm 会引发“此字段不能为空”验证错误。

我真的没有发现我做错了什么，因为我在 ModelForm 的 clean_contact() 方法中使用 get_or_create() 方法来插入此联系人(如果需要)，然后返回结果对象来更新 self.cleaned_data 字典。最糟糕的是，当我第二次提交表单而不更改任何字段时，一切都运行顺利(不再有验证错误)...

当我使用数据库中现有的邮件地址提交表单时，第一次提交表单时一切正常。

如果你们能帮助指出我的代码中有什么问题以及为什么会在每个发布的数据都是正确的情况下引发此错误，我将非常感激。

但是我对 AJAX mixin 与我的 CreateView 有一点疑问，因为也许当邮件地址未知时， get_or_create() 创建并返回它，但是 - 我无法想象为什么 - 创建Alert 对象尚无法引用新创建的 Contact 对象。这可以解释为什么第二次提交有效...我相信你们会得到最终决定:-)

下面涉及上述问题的不同应用程序部分。我自愿删除了一些未用于此应用程序的 Model 字段以及从 CreateView 继承的 LoginRequiredMixin。

我再次非常感谢您对此的帮助，并提前感谢您提出的每一条建议。

型号

class UmsAlerting(models.Model):
    alert_id = models.IntegerField(primary_key=True, editable=False)
    appli = models.ForeignKey('UmsApplication')
    env = models.ForeignKey('UmsEnvironment')
    contact = models.ForeignKey('UmsContacts')
    class Meta:
        db_table = 'ums_alerting'
    def __unicode__(self):
        return u'Alert_Id %d on %s(%s)' %(self.alert_id, self.appli.trigram_ums, self.env.env_name)


class UmsApplication(models.Model):
    appli_id = models.IntegerField(primary_key=True)
    trigram_ums = models.CharField(max_length=4L)
    class Meta:
        db_table = 'ums_application'


class UmsContacts(models.Model):
    contact_id = models.IntegerField(primary_key=True)
    mail_addr = models.CharField(max_length=100L)
    class Meta:
        db_table = 'ums_contacts'


class UmsEnvironment(models.Model):
    env_id = models.IntegerField(primary_key=True)
    env_name = models.CharField(max_length=5L)
    class Meta:
        db_table = 'ums_environment'

    def __unicode__(self):
        return self.env_name

模型表单

class AlertForm(ModelForm):
    class Meta:
        model = UmsAlerting
        exclude = ('custom_rule')

    appli = forms.CharField(required=True, max_length=3)
    env = forms.ModelChoiceField(required=True,
                                 queryset=UmsEnvironment.objects.all())
    contact = forms.EmailField(required=True)

    def clean_appli(self):
        data = self.cleaned_data['appli']

        try:
            UmsApplication.objects.get(trigram_ums=data)
        except ObjectDoesNotExist:
            msg = 'Trigram must be known and valid.'
            self._errors['appli'] = self.error_class([msg])
            raise forms.ValidationError(msg)

        return UmsApplication.objects.get(trigram_ums=data)

    def clean_contact(self):
        data = self.cleaned_data['contact']
        c, created = UmsContacts.objects.get_or_create(mail_addr=data)

        return c


    def clean(self):
        cleaned_data = super(AlertForm, self).clean()
        app = cleaned_data.get('appli')
        contact = cleaned_data.get('contact')
        env = cleaned_data.get('env')

        # Do not insert a new alert if it already exists
        if UmsAlerting.objects.filter(appli=app, env=env, contact=contact).count() > 0:
            msg = 'Alert is already configured.'
            self._errors['contact'] = self.error_class([msg])
            raise forms.ValidationError(msg)

        # Return the parent's clean method finally
        return cleaned_data

创建 View

class AlertView(LoginRequiredMixin, AjaxResponseMixin, CreateView):
     template_name = 'tools/alert_form.html'
     form_class = AlertForm
     success_url = reverse_lazy('alerts_configure')

AjaxResponseMixin

class AjaxResponseMixin(object):
     def render_to_json_response(self, context, **kwargs):
         data = json.dumps(context)
         kwargs['content_type'] = 'application/json'
         return HttpResponse(data, **kwargs)

     def form_invalid(self, form):
         response = super(AjaxResponseMixin, self).form_invalid(form)
         if self.request.is_ajax():
             return self.render_to_json_response(form.errors, status=400)
         else:
             return response

     # Not really useful actually (yet)
     def form_valid(self, form):
         response = super(AjaxResponseMixin, self).form_valid(form)
         if self.request.is_ajax():
             return self.render_to_json_response(json.dumps({}))
         else:
             return response

Answer 1

答案已描述here

简而言之，UmsContacts 模型使用 IntegerField，因此必须指定它来创建像这样的新对象，即使自动增量设置为数据库表。解决方案是将其更改为 AutoField 并在 clean_contact 方法中返回修改后的 self.cleaned_data['contact']。