#34590: Querying for decimals larger than max_digits crashes on SQLite
-------------------------------------+-------------------------------------
     Reporter:  Marc Odermatt        |                    Owner:  Mohit
                                     |  Singh Sinsniwal
         Type:  Bug                  |                   Status:  assigned
    Component:  Database layer       |                  Version:  4.2
  (models, ORM)                      |
     Severity:  Release blocker      |               Resolution:
     Keywords:                       |             Triage Stage:  Accepted
    Has patch:  0                    |      Needs documentation:  0
  Needs tests:  0                    |  Patch needs improvement:  0
Easy pickings:  0                    |                    UI/UX:  0
-------------------------------------+-------------------------------------

Comment (by Florian Apolloner):

 Mhm this is a tricky one. Raising `EmptyResultSet` feels wrong though.
 There is no reason why this code should not be able to properly prep the
 value (there will certainly be dragons somewhere down the line but it
 feels like we are doing one transformation to many). I'd be interested to
 know what the actual code was that led to this behavior. If you pass
 `.filter(pk='abc')` you also get an exception that this is not a valid
 integer. Wouldn't form validation where the validator is derived from the
 model field would have detected that this is out of range already?

-- 
Ticket URL: <https://code.djangoproject.com/ticket/34590#comment:5>
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 view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/010701884425524e-ead891da-b6d3-4f7c-9d41-9018042e0170-000000%40eu-central-1.amazonses.com.

Reply via email to