Commit 8badb758 authored by Karen Tracey's avatar Karen Tracey
Browse files

[1.1.X] Fixed #12239: Fixed results of gte and lt queries when comparing... 

[1.1.X] Fixed #12239: Fixed results of gte and lt queries when comparing floats to integer field values.
Thanks waverider, aroy, SmileyChris, and pmclanahan. 


git-svn-id: http://code.djangoproject.com/svn/django/branches/releases/1.1.X@12820 bcc190cf-cafb-0310-a4f2-bffc1f526a37
parent 4d00ac86

django/db/models/fields/__init__.py

+6 −0
Original line number Diff line number Diff line
@@ -2,6 +2,7 @@ import datetime 
import os

import re

import time

import math



import django.utils.copycompat as copy
@@ -722,6 +723,11 @@ class IntegerField(Field): 
            return None

        return int(value)



    def get_db_prep_lookup(self, lookup_type, value):

        if lookup_type == 'gte' or lookup_type == 'lt':

            value = math.ceil(value)

        return super(IntegerField, self).get_db_prep_lookup(lookup_type, value)



    def get_internal_type(self):

        return "IntegerField"

tests/regressiontests/queries/models.py

+39 −0
Original line number Diff line number Diff line
@@ -417,6 +417,45 @@ constraints. 
>>> Number.objects.filter(Q(num__gt=7) & Q(num__lt=12) | Q(num__lt=4))

[<Number: 8>]



Bug #12239

Float was being rounded to integer on gte queries on integer field.  Tests

show that gt, lt, gte, and lte work as desired.  Note that the fix changes

get_prep_lookup for gte and lt queries only.

>>> Number.objects.filter(num__gt=11.9)

[<Number: 12>]

>>> Number.objects.filter(num__gt=12)

[]

>>> Number.objects.filter(num__gt=12.0)

[]

>>> Number.objects.filter(num__gt=12.1)

[]

>>> Number.objects.filter(num__lt=12)

[<Number: 4>, <Number: 8>]

>>> Number.objects.filter(num__lt=12.0)

[<Number: 4>, <Number: 8>]

>>> Number.objects.filter(num__lt=12.1)

[<Number: 4>, <Number: 8>, <Number: 12>]

>>> Number.objects.filter(num__gte=11.9)

[<Number: 12>]

>>> Number.objects.filter(num__gte=12)

[<Number: 12>]

>>> Number.objects.filter(num__gte=12.0)

[<Number: 12>]

>>> Number.objects.filter(num__gte=12.1)

[]

>>> Number.objects.filter(num__gte=12.9)

[]

>>> Number.objects.filter(num__lte=11.9)

[<Number: 4>, <Number: 8>]

>>> Number.objects.filter(num__lte=12)

[<Number: 4>, <Number: 8>, <Number: 12>]

>>> Number.objects.filter(num__lte=12.0)

[<Number: 4>, <Number: 8>, <Number: 12>]

>>> Number.objects.filter(num__lte=12.1)

[<Number: 4>, <Number: 8>, <Number: 12>]

>>> Number.objects.filter(num__lte=12.9)

[<Number: 4>, <Number: 8>, <Number: 12>]



Bug #7872

Another variation on the disjunctive filtering theme.