Pourquoi le model.save () de django n'appelle-t-il pas full_clean ()?

Je suis juste curieux de savoir s'il y a une bonne raison pour laquelle orm de django n'appelle pas 'full_clean' sur un modèle à moins qu'il ne soit enregistré dans le cadre d'un formulaire de modèle.

Notez que full_clean () ne sera pas appelé automatiquement lorsque vous appelez la méthode save () de votre modèle. Vous devrez l'appeler manuellement lorsque vous souhaitez exécuter la validation de modèle en une étape pour vos propres modèles créés manuellement. Doc propre complet de django

(NOTE: citation mise à jour pour Django 1.6 ... les précédents documents django avaient également une mise en garde concernant ModelForms.)

Y a-t-il de bonnes raisons pour lesquelles les gens ne voudraient pas ce comportement? Je pense que si vous preniez le temps d'ajouter une validation à un modèle, vous voudriez que cette validation soit exécutée à chaque fois que le modèle est enregistré.

Je sais comment tout faire fonctionner correctement, je cherche juste une explication.

AFAIK, c'est à cause de la rétrocompatibilité. Il y a aussi des problèmes avec ModelForms avec des champs exclus, des modèles avec des valeurs par défaut, des signaux pre_save (), etc.

Sources qui pourraient vous intéresser:

• http://code.djangoproject.com/ticket/13100
• http://groups.google.com/group/django-developers/browse_frm/thread/b888734b05878f87

En raison de la compatibilité, le nettoyage automatique à la sauvegarde n'est pas activé dans le noyau django.

Si nous commençons un nouveau projet et que nous voulons que la saveméthode par défaut sur Model puisse nettoyer automatiquement, nous pouvons utiliser le signal suivant pour effectuer un nettoyage avant que chaque modèle ne soit enregistré.

from django.dispatch import receiver
from django.db.models.signals import pre_save, post_save

@receiver(pre_save)
def pre_save_handler(sender, instance, *args, **kwargs):
    instance.full_clean()

Le moyen le plus simple d'appeler la full_cleanméthode consiste simplement à remplacer la saveméthode dans votre model:

def save(self, *args, **kwargs):
    self.full_clean()
    return super(YourModel, self).save(*args, **kwargs)

Au lieu d'insérer un morceau de code qui déclare un récepteur, nous pouvons utiliser une application comme INSTALLED_APPSsection danssettings.py

INSTALLED_APPS = [
    # ...
    'django_fullclean',
    # your apps here,
]

Avant cela, vous devrez peut-être installer à l' django-fullcleanaide de PyPI:

pip install django-fullclean

Si vous avez un modèle dont vous voulez vous assurer qu'il a au moins une relation FK, et que vous ne voulez pas utiliser null=Falseparce que cela nécessite la définition d'un FK par défaut (qui seraient des données inutiles), la meilleure façon que j'ai trouvée est pour ajouter de la personnalisation .clean()et des .save()méthodes. .clean()lève l'erreur de validation et .save()appelle le fichier clean. De cette façon, l'intégrité est appliquée à la fois à partir des formulaires et à partir d'autres codes d'appel, de la ligne de commande et des tests. Sans cela, il n'y a (AFAICT) aucun moyen d'écrire un test qui garantit qu'un modèle a une relation FK avec un autre modèle spécifiquement choisi (pas par défaut).

class Payer(models.Model):

    name = models.CharField(blank=True, max_length=100)
    # Nullable, but will enforce FK in clean/save:
    payer_group = models.ForeignKey(PayerGroup, null=True, blank=True,)

    def clean(self):
        # Ensure every Payer is in a PayerGroup (but only via forms)
        if not self.payer_group:
            raise ValidationError(
                {'payer_group': 'Each Payer must belong to a PayerGroup.'})

    def save(self, *args, **kwargs):
        self.full_clean()
        return super().save(*args, **kwargs)

    def __str__(self):
        return self.name

Commenter la réponse de @Alfred Huang et commenter. On peut verrouiller le hook pre_save sur une application en définissant une liste de classes dans le module actuel (models.py) et en la comparant dans le hook pre_save:

CUSTOM_CLASSES = [obj for name, obj in
        inspect.getmembers(sys.modules[__name__])
        if inspect.isclass(obj)]

@receiver(pre_save)
def pre_save_handler(sender, instance, **kwargs):
    if type(instance) in CUSTOM_CLASSES:
        instance.full_clean()