Django F sembra non funzionare?

Question

Huh, per qualche ragione non riesco a far funzionare correttamente la F anche sui modelli più semplici. Qui su Django 1.9.x.

Nella forma più semplice, Testaccount

class TestAccount(models.Model): 
    decimal = models.DecimalField(max_digits=5, decimal_places=2) 
    integer = models.IntegerField() 



In [1]: ta = TestAccount() 

In [2]: ta.integer = 1 

In [3]: ta.decimal = 1 

In [4]: ta.save() 

In [5]: 

In [5]: 

In [5]: ta 
Out[5]: <TestAccount: TestAccount object> 

In [6]: ta.id 
Out[6]: 1L 

In [7]: from django.db.models.expressions import F 

In [8]: ta = TestAccount.objects.get(id=1) 

In [9]: ta.integer = F('integer') + 1 

In [10]: ta.save() 
--------------------------------------------------------------------------- 
ValidationError       Traceback (most recent call last) 
<ipython-input-10-6e9eda341b34> in <module>() 
----> 1 ta.save() 

/usr/lib/python2.7/site-packages/django/db/models/base.pyc in save(self, force_insert, force_update, using, update_fields) 
    706 
    707   self.save_base(using=using, force_insert=force_insert, 
--> 708      force_update=force_update, update_fields=update_fields) 
    709  save.alters_data = True 
    710 

/usr/lib/python2.7/site-packages/django/db/models/base.pyc in save_base(self, raw, force_insert, force_update, using, update_fields) 
    730   if not meta.auto_created: 
    731    signals.pre_save.send(sender=origin, instance=self, raw=raw, using=using, 
--> 732         update_fields=update_fields) 
    733   with transaction.atomic(using=using, savepoint=False): 
    734    if not raw: 

/usr/lib/python2.7/site-packages/django/dispatch/dispatcher.pyc in send(self, sender, **named) 
    190 
    191   for receiver in self._live_receivers(sender): 
--> 192    response = receiver(signal=self, sender=sender, **named) 
    193    responses.append((receiver, response)) 
    194   return responses 

/media/sf_helium/build/helium/internal/signals.pyc in validate_model(sender, **kwargs) 
    12 def validate_model(sender, **kwargs): 
    13  if 'raw' in kwargs and not kwargs['raw']: 
---> 14   kwargs['instance'].full_clean() 
    15 
    16 @receiver(pre_delete) 

/usr/lib/python2.7/site-packages/django/db/models/base.pyc in full_clean(self, exclude, validate_unique) 
    1142 
    1143   if errors: 
-> 1144    raise ValidationError(errors) 
    1145 
    1146  def clean_fields(self, exclude=None): 

ValidationError: {'integer': [u"'F(integer) + Value(1)' value must be an integer."]} 

Ma secondo questo: https://docs.djangoproject.com/en/1.9/ref/models/instances/#updating-attributes-based-on-existing-fields dovrebbe funzionare ...

Perché F non viene esclusa dalla convalida non lo so. Dovrebbe essere, e Django dovrebbe semplicemente creare una query per aggiornarlo.

Answer 1

Questo funziona bene in puro Django. Il problema nel tuo caso è che avete un ascoltatore (in helium.internal.signals) al segnale pre_save che cerca di fare questo:

def validate_model(sender, **kwargs): 
    if 'raw' in kwargs and not kwargs['raw']: 
     kwargs['instance'].full_clean() 

Model.full_clean aspetta un gruppo di valori per ogni campo nel modello, ma in questo caso si dei tuoi campi non è un valore ma uno CombinedExpression che non è stato ancora valutato e verrà valutato solo quando Django scrive nel database. Questo causa l'errore.

IMO vi sia bisogno di eseguire il proprio convalida che implementa la logica della full_clean e gestisce Expression s, oppure è necessario escludere i campi contenenti espressioni da full_clean.