IntegrityError(错误)而不是 ValidationError(正确)

IntegrityError (wrong) instead of ValidationError (correct)

Django 版本:3.1.5

Python版本:3.6.9

型号

class GoogleAnalytics(models.Model):
    counter = models.SlugField(max_length=17,
                               null=False,
                               default="",
                               unique=True,
                               verbose_name="Counter")

    tracking_code = models.TextField(null=False,
                                     default="",
                                     verbose_name="Tracking code",
                                     validators=[validate_google_analytics_tracking_code])

表格

class CounterForm(forms.ModelForm):
    """
    Validate if the value in counter field
    really corresponds to the tracking code for this counter.
    """

    def clean(self):
        counter_manager = CounterManager()
        if "GoogleAnalytics" in str(self.Meta.model):
            counter_from_tracking = counter_manager.get_counter(self.cleaned_data['tracking_code'],
                                                                Counters.GOOGLE_ANALYTICS)
        else:
            assert "YandexMetrika" in str(self.Meta.model)
            counter_from_tracking = counter_manager.get_counter(self.cleaned_data['tracking_code'],
                                                                Counters.YANDEX_METRIKA)
        if self.cleaned_data['counter'] != counter_from_tracking:
            raise ValidationError("Код не соответствует счетчику")

        return self.cleaned_data

    class Meta:
        model = GoogleAnalytics
        exclude = []

管理员

class GoogleAnalyticsAdmin(admin.ModelAdmin):
    form = CounterForm


admin.site.register(GoogleAnalytics, GoogleAnalyticsAdmin)

回溯

https://dpaste.com/6CUL2VZAW

嗯,计数器字段的唯一约束有效。

但我检测到的是验证错误而不是 IntegrityError。你能告诉我如何解决这个问题吗?

我认为您的问题是您覆盖了 clean 方法并且没有调用它的依赖项。因为清理你的超级模型表单,所以在点击并在数据库中写入新记录之前验证你的唯一约束。所以我认为将行 super().clean() 添加到您的 clean 方法应该可以解决您的问题。

def clean(self):
    super().clean()
    counter_manager = CounterManager()
    if "GoogleAnalytics" in str(self.Meta.model):
        counter_from_tracking = counter_manager.get_counter(self.cleaned_data['tracking_code'],
                                                            Counters.GOOGLE_ANALYTICS)
    else:
        assert "YandexMetrika" in str(self.Meta.model)
        counter_from_tracking = counter_manager.get_counter(self.cleaned_data['tracking_code'],
                                                            Counters.YANDEX_METRIKA)
    if self.cleaned_data['counter'] != counter_from_tracking:
        raise ValidationError("Код не соответствует счетчику")

    return self.cleaned_data

或者如果您想在 super 条件之前检查您的条件,您可以将 super().clean() 移动到 clean 方法的最后一行。