#25425: Enforce calling resolve_expression before as_sql on all expressions
-------------------------------------+-------------------------------------
     Reporter:  jarshwah             |                    Owner:  nobody
         Type:  Bug                  |                   Status:  new
    Component:  Database layer       |                  Version:  master
  (models, ORM)                      |
     Severity:  Normal               |               Resolution:
     Keywords:                       |             Triage Stage:
                                     |  Unreviewed
    Has patch:  0                    |      Needs documentation:  0
  Needs tests:  0                    |  Patch needs improvement:  0
Easy pickings:  0                    |                    UI/UX:  0
-------------------------------------+-------------------------------------

Comment (by akaariai):

 We have two interfaces:
   - Resolvable: resolved to an expression with resolve_expression
   - Compilable: can be turned into SQL

 Not all compilables are resolvables, nor the other way around. So, it
 doesn't make sense to enforce that all compilables have been resolved.

 I'm not even sure what WhereNode.resolve_expression() would do? We have
 other similar types that are not resolved ever, for example BaseTable and
 Join.

 We could do the check for those cases where we have the resolve + compile
 contract.

--
Ticket URL: <https://code.djangoproject.com/ticket/25425#comment:1>
Django <https://code.djangoproject.com/>
The Web framework for perfectionists with deadlines.

-- 
You received this message because you are subscribed to the Google Groups 
"Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to [email protected].
To post to this group, send email to [email protected].
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/066.dace88662c872641aa84d54ed10f0d66%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.

Reply via email to