Perché il modello.save () di django non chiama full_clean ()?

Sono solo curioso di sapere se qualcuno sa se c'è una buona ragione per cui orm di django non chiama 'full_clean' su un modello a meno che non venga salvato come parte di un modulo modello.

Si noti che full_clean () non verrà chiamato automaticamente quando si chiama il metodo save () del modello. Dovrai chiamarlo manualmente quando desideri eseguire la convalida del modello in un passaggio per i tuoi modelli creati manualmente. django è un documento completamente pulito

(NOTA: la citazione aggiornata per Django 1.6 ... i precedenti documenti di django avevano anche un avvertimento su ModelForms.)

Ci sono buoni motivi per cui le persone non vogliono questo comportamento? Penserei che se ti prendessi il tempo per aggiungere la convalida a un modello, vorresti che quella convalida venga eseguita ogni volta che il modello viene salvato.

So come far funzionare tutto correttamente, sto solo cercando una spiegazione.

AFAIK, questo è dovuto alla compatibilità con le versioni precedenti. Ci sono anche problemi con ModelForms con campi esclusi, modelli con valori predefiniti, segnali pre_save (), ecc.

Fonti in cui potresti essere interessato:

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

A causa della compatibilità considerata, la pulizia automatica al salvataggio non è abilitata nel kernel di django.

Se stiamo avviando un nuovo progetto e desideriamo che il savemetodo predefinito su Modello possa essere pulito automaticamente, possiamo utilizzare il seguente segnale per eseguire la pulizia prima di salvare ogni modello.

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()

Il modo più semplice per chiamare il full_cleanmetodo è solo quello di sovrascrivere il savemetodo in model:

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

Invece di inserire un pezzo di codice che dichiara un destinatario, possiamo usare un'app come INSTALLED_APPSsezione insettings.py

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

Prima di ciò, potrebbe essere necessario installare django-fullcleanutilizzando PyPI:

pip install django-fullclean

Se si dispone di un modello che si desidera garantire abbia almeno una relazione FK e non si desidera utilizzare null=Falseperché ciò richiede l'impostazione di un FK predefinito (che sarebbe dati inutili), il modo migliore che ho trovato è per aggiungere personalizzazioni .clean()e .save()metodi. .clean()genera l'errore di convalida e .save()chiama clean. In questo modo l'integrità viene applicata sia dai moduli sia da altro codice chiamante, dalla riga di comando e dai test. Senza questo, non esiste (AFAICT) un modo per scrivere un test che assicuri che un modello abbia una relazione FK con un altro modello scelto (non predefinito).

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

Commentando la risposta di @Alfred Huang e la commenta. Si potrebbe bloccare l'hook pre_save in basso su un'app definendo un elenco di classi nel modulo corrente (models.py) e verificando contro di esso nell'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()