Как следовать подсказке: используйте взамен вызываемый, например, используйте `dict` вместо` {} `?

Question

Как соблюдать предупреждения?

models.py

from django.contrib.postgres.fields import JSONField
from django.db import models
from django_extensions.db.models import TimeStampedModel


class UnderwritingValidator(TimeStampedModel):
    plan = models.PositiveIntegerField(null=True, blank=True, unique=True)
    logic = JSONField(default=dict(
        accept_list=[],
        reject_list=[]
    ))

Затем makemigrations

WARNINGS:
uw_validators.UnderwritingValidator.logic: (postgres.E003) JSONField default should be a callable instead of an instance so that it's not shared between all field instances.
    HINT: Use a callable instead, e.g., use `dict` instead of `{}`.
Migrations for 'uw_validators':
  uw_validators/migrations/0002_auto_20191011_0321.py
    - Remove field accept_list from underwritingvalidator
    - Remove field reject_list from underwritingvalidator
    - Add field logic to underwritingvalidator

Программное обеспечение:

postgres: 10,9
Django == 2.2.5

Вопросы:

1. Я в безопасности от ошибок? Если это безопасно, я проигнорирую это предупреждение и сотру короткую заметку с предупреждением
2. Как полностью следовать предупреждению?

Answer 1

Это не для вызова.

У вас есть два варианта здесь:

1. Положитесь на dict по умолчанию;это приведет к тому, что ваши модели будут использовать пустой dict {}, если ни один не указан:

class UnderwritingValidator(TimeStampedModel):
    plan = models.PositiveIntegerField(null=True, blank=True, unique=True)
    logic = JSONField(default=dict)

Создайте свой собственный «вызываемый» и используйте его по умолчанию:

def get_default_something():
    return {'accept_list': [], 'reject_list': []}

class UnderwritingValidator(TimeStampedModel):
    plan = models.PositiveIntegerField(null=True, blank=True, unique=True)
    logic = JSONField(default=get_default_something)