如何在 Django Admin 中向用户显示数据库错误

Question

背景：我的 Django 应用程序位于预先存在的 Postgresql 数据库之上。 这个数据库有一个非常复杂的触发器和约束网络。

问题：在 Django Admin 中，如果用户在保存时导致数据库错误，我想以用户友好的格式向他们显示错误，类似于内置的 forms.ValidationError。

示例（这不起作用，它会导致 500）：

def save_model(self, request, obj, form, change):
    try:
        obj.save()
    except DatabaseError as e:
        raise forms.ValidationError(e)

预期结果：

在管理员中向用户显示，“ Database Error: ID 58574 - Price is outside customers requested range. Cannot add or update a child row: a foreign key constraint fails 。”

Answer 1

如果可能，您需要略微更改逻辑。 您需要的是自定义AdminModel.form 。 所有验证都应该在那里进行。 请参阅save_model()的注释：

ModelAdmin.save_model（）和ModelAdmin.delete_model（）必须保存/删除对象，它们不是用于否决目的，而是允许您执行额外的操作。

但是如果你的情况不能在表格中进行所有验证，我将ModelAdmin并覆盖def add_view() ， def change_view()和def changelist_view()就像这样：

from django.contrib import admin
from django import forms
from django.contrib.admin import helpers
from django.contrib.admin.options import csrf_protect_m, IS_POPUP_VAR
from django.utils.translation import ugettext as _
from django.utils.encoding import force_text

# for nonfield errors to show correctly
from django.forms.forms import NON_FIELD_ERRORS

from .models import TestModel


class TestModelAdmin(admin.ModelAdmin):

    def save_model(self, request, obj, form, change):

        raise Exception('test exception')

    @csrf_protect_m
    def add_view(self, request, form_url='', extra_context=None):
        try:
            return super(TestModelAdmin, self).add_view(request, form_url, extra_context)
        except Exception as e:
            pass

        # mimic parent class on error

        model = self.model
        opts = model._meta

        ModelForm = self.get_form(request)
        formsets = []
        inline_instances = self.get_inline_instances(request, None)
        form = ModelForm(request.POST, request.FILES)
        form.is_valid()

        # make faked nonfield error
        # see http://stackoverflow.com/questions/8598247/how-to-append-error-message-to-form-non-field-errors-in-django
        form._errors[NON_FIELD_ERRORS] = form.error_class([e.message])

        # We may handle exception here (just to save indentation)
        adminForm = helpers.AdminForm(form, list(self.get_fieldsets(request)),
            self.get_prepopulated_fields(request),
            self.get_readonly_fields(request),
            model_admin=self)
        media = self.media + adminForm.media

        inline_admin_formsets = []
        for inline, formset in zip(inline_instances, formsets):
            fieldsets = list(inline.get_fieldsets(request))
            readonly = list(inline.get_readonly_fields(request))
            prepopulated = dict(inline.get_prepopulated_fields(request))
            inline_admin_formset = helpers.InlineAdminFormSet(inline, formset,
                fieldsets, prepopulated, readonly, model_admin=self)
            inline_admin_formsets.append(inline_admin_formset)
            media = media + inline_admin_formset.media

        context = {
            'title': _('Add %s') % force_text(opts.verbose_name),
            'adminform': adminForm,
            'is_popup': IS_POPUP_VAR in request.REQUEST,
            'media': media,
            'inline_admin_formsets': inline_admin_formsets,
            'errors': helpers.AdminErrorList(form, formsets),
            'app_label': opts.app_label,
            'preserved_filters': self.get_preserved_filters(request),
        }
        context.update(extra_context or {})
        return self.render_change_form(request, context, form_url=form_url, add=True)

admin.site.register(TestModel, TestModelAdmin)

我的models.py ：

from django.db import models

class TestModel(models.Model):

    text = models.TextField()

你看，在save_model()里面没有简单的方法可以挂钩，所以你必须复制粘贴部分表格准备代码。

Answer 2

@twil - 感谢您的帮助。 你让我走上了正确的轨道。 非常感谢您的帮助。 但是，解决方案没有开箱即用。 实际上没有在我的测试用例中显示错误或使用change_view。 这是我希望我最终合作。

from django.contrib.admin import ModelAdmin
from django.db import DatabaseError, IntegrityError
from django.contrib import messages


class ShowValidationAdmin(ModelAdmin):

    def add_view(self, request, form_url='', extra_context=None):
        try:
            return super(ShowValidationAdmin, self).add_view(request, form_url, extra_context)
        except (IntegrityError, DatabaseError) as e:

            request.method = 'GET'
            messages.error(request, e.message)
            return super(ShowValidationAdmin, self).add_view(request, form_url, extra_context)

    def change_view(self, request, object_id, form_url='', extra_context=None):
        try:
            return super(ShowValidationAdmin, self).change_view(request, object_id, form_url, extra_context)
        except (IntegrityError, DatabaseError) as e:

            request.method = 'GET'
            messages.error(request, e.message)
            return super(ShowValidationAdmin, self).change_view(request, object_id, form_url, extra_context)

注意：这个版本似乎也适用于交叉版本（django 1.3 - 1.6）。 如果有人有更好的方法，请告诉我。 我等着奖励赏金。

Answer 3

尝试这个：

 from django.core.exceptions import ValidationError
    def save_model(self, request, obj, form, change):
        try:
            obj.save()
        except DatabaseError as e:
            raise ValidationError(e)

Answer 4

老问题，但还没有人提到ModelAdmin.message_user() ，它也使用 消息框架。

例如：

from django.contrib import admin, messages

class MyModelAdmin(admin.ModelAdmin):
    ...
    
    def save_model(self, request, *args, **kwargs):
        try:
            super().save_model(request, *args, **kwargs)
        except DatabaseError as e:
            messages.set_level(request=request, level=messages.ERROR)            
            self.message_user(request=request, message=e, level=messages.ERROR)

这将显示您的消息如下（假设这是e中的错误字符串）：

set_level调用会抑制默认的成功消息。 另请参见此处和此处。

Answer 5

在save_model()中，您可以单独和不同地处理DatabaseError异常以添加或更改 object ，如下所示：

@admin.register(Person)
class PersonAdmin(admin.ModelAdmin):
    
    def save_model(self, request, obj, form, change): # Here
        last_part_of_path = request.path.split('/')[-2]

        if last_part_of_path == "add":
            try: # ↓ I intentionally raise an exception
                raise DatabaseError("Add Error")
                obj.save()
            except DatabaseError as e:
                messages.set_level(request, messages.ERROR)
                messages.error(request, e)

        if last_part_of_path == "change":
            try: # ↓ I intentionally raise an exception
                raise DatabaseError("Change Error")
                obj.save()
            except DatabaseError as e:
                messages.set_level(request, messages.ERROR)
                messages.error(request, e)