diff options
author | Bakdolot <80908236+Bakdolot@users.noreply.github.com> | 2023-04-24 17:14:35 +0600 |
---|---|---|
committer | GitHub <noreply@github.com> | 2023-04-24 08:14:35 -0300 |
commit | c813fb327cb1b09542be89c5ceed367826236bc2 (patch) | |
tree | 94f90414368cc6f962fcfb3f7035a1915a92c357 /django | |
parent | 83c9765f45e4622e4a5af3adcd92263a28b13624 (diff) | |
download | django-c813fb327cb1b09542be89c5ceed367826236bc2.tar.gz |
Fixed #34481 -- Added system check for reverse related fields in ModelAdmin.list_display.
Diffstat (limited to 'django')
-rw-r--r-- | django/contrib/admin/checks.py | 7 |
1 files changed, 5 insertions, 2 deletions
diff --git a/django/contrib/admin/checks.py b/django/contrib/admin/checks.py index 27537d9614..0fefe65e48 100644 --- a/django/contrib/admin/checks.py +++ b/django/contrib/admin/checks.py @@ -916,10 +916,13 @@ class ModelAdminChecks(BaseModelAdminChecks): id="admin.E108", ) ] - if isinstance(field, models.ManyToManyField): + if isinstance(field, models.ManyToManyField) or ( + getattr(field, "rel", None) and field.rel.field.many_to_one + ): return [ checks.Error( - "The value of '%s' must not be a ManyToManyField." % label, + f"The value of '{label}' must not be a many-to-many field or a " + f"reverse foreign key.", obj=obj.__class__, id="admin.E109", ) |