Naming a model Check causes Django's check framework to fail with an exception

[Bram Duvigneau]

This causes an SingleRelatedObjectDescriptor is not callable error.

[Collin Anderson]

I can't reproduce this. Is the Check model related to another model? Could you post a traceback?

[Baptiste Mispelon]

I've managed to reproduce with these models:

class Foo(models.Model):
    pass


class Bar(models.Model):
    check = models.ForeignKey('Foo')

It turns out that the name of the model is not the issue but the name of the field is.
Note however that using check = models.IntegerField() doesn't trigger the error.

[Rigel Di Scala]

I can replicate the issue.

The exception is raised, when running my test using the Django test management command, here: ​https://github.com/django/django/blob/master/django/core/management/base.py#L418

 if (self.requires_system_checks and
                    not options.get('skip_validation') and  # Remove at the end of deprecation for `skip_validation`.
                    not options.get('skip_checks')):
                self.check()

bmispelon and I found that the problem actually originates here:
​https://github.com/django/django/blob/master/django/core/checks/model_checks.py#L14

@register(Tags.models)
def check_all_models(app_configs=None, **kwargs):
    errors = [model.check(**kwargs)
        for model in apps.get_models()
        if app_configs is None or model._meta.app_config in app_configs]
    return list(chain(*errors))

We would therefore need to check (no pun intended) that the check attribute on the model instance does not resolve to a field, before actually calling it as a method, something to the tune of:

if mdl._meta.get_field_by_name("check"):
    pass

[Rigel Di Scala]

I have submitted a pull request that amends the behavior of check_all_models(), so that is checks for this problem and returns a meaningful error message.

See: ​https://github.com/django/django/pull/3362

[Loic Bistuer]

@zedr thanks for the patch. In term of procedure though we only mark a ticket as fixed once a patch has been merged. As a matter of fact, tickets close themselves automatically provided the commit messages have the right format.

[Rigel Di Scala]

@loic I see. Sorry about that.

[Collin Anderson]

It seems like we're treating this like release blocker.

[Loic Bistuer <loic.bistuer@…>]

In a5c77417a651c93036cf963e6da518653115be7e:

Fixed #23615 -- Validate that a Model instance's "check" attribute is a method.

The "check" name is a reserved word used by Django's check framework,
and cannot be redefined as something else other than a method, or the check
framework will raise an error.

This change amends the django.core.checks.model_check.check_all_models()
function, so that it verifies that a model instance's attribute "check"
is actually a method. This new check is assigned the id "models.E020".

[Loic Bistuer <loic.bistuer@…>]

In e8262b59418d68d55a97a25567e4bc4f078ed3cb:

[1.7.x] Fixed #23615 -- Validate that a Model instance's "check" attribute is a method.

The "check" name is a reserved word used by Django's check framework,
and cannot be redefined as something else other than a method, or the check
framework will raise an error.

This change amends the django.core.checks.model_check.check_all_models()
function, so that it verifies that a model instance's attribute "check"
is actually a method. This new check is assigned the id "models.E020".

Conflicts:

docs/ref/checks.txt

Backport of a5c77417a6 from master