#24385: Problem with aggregate(Sum())
----------------------------------------------+--------------------
     Reporter:  mcagl                         |      Owner:  nobody
         Type:  Uncategorized                 |     Status:  new
    Component:  Database layer (models, ORM)  |    Version:  1.6
     Severity:  Normal                        |   Keywords:
 Triage Stage:  Unreviewed                    |  Has patch:  0
Easy pickings:  0                             |      UI/UX:  0
----------------------------------------------+--------------------
 Hello,
 I setup a minimal project here:
 https://github.com/mcagl/minimal_django_sum_test to demonstrate the
 problem.

 I'm working with Django 1.6.x and I noticed something that I don't
 understand.

 As you can see from the github repository, I have a Tag model and a Row
 model with a m2m towards Tag and a DecimalField called amount.

 If I filter Row objects for more than one Tag, and there is/are Row
 objects that have more than one Tag among the one filtered by,
 Sum('amount') counts it/them once per Tag, even if I use distinct().

 Please note also that I assert, in the test, that the filtered queryset is
 composed by three Row objects, as expected, but the next assert fails,
 with 40 != 30.

 I added a test that instead of aggregate(Sum('amount')) does sum([x.amount
 for x in rows]) which passes.

 Is this a bug in Sum() or am I missing something?

 Kind regards,
 Mark

--
Ticket URL: <https://code.djangoproject.com/ticket/24385>
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/048.f395a8303007392a075f0ec01e045fd5%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.

Reply via email to