From 740c97454167a17fbccedc6e775f3bb85904ec01 Mon Sep 17 00:00:00 2001 From: Karen Tracey Date: Sun, 21 Mar 2010 20:54:24 +0000 Subject: [1.1.X] Fixed #12822: Don't copy the _aggregate_select_cache when cloning a query set, as that can lead to incorrect SQL being generated for the query. Thanks to mat for the report and test, tobias for the fix, and Alex for review. r12830 from trunk. git-svn-id: http://code.djangoproject.com/svn/django/branches/releases/1.1.X@12831 bcc190cf-cafb-0310-a4f2-bffc1f526a37 --- django/db/models/sql/query.py | 10 ++++++---- 1 file changed, 6 insertions(+), 4 deletions(-) (limited to 'django/db/models/sql') diff --git a/django/db/models/sql/query.py b/django/db/models/sql/query.py index 6b9f4b6d56..f09dc313bc 100644 --- a/django/db/models/sql/query.py +++ b/django/db/models/sql/query.py @@ -215,10 +215,12 @@ class BaseQuery(object): obj.aggregate_select_mask = None else: obj.aggregate_select_mask = self.aggregate_select_mask.copy() - if self._aggregate_select_cache is None: - obj._aggregate_select_cache = None - else: - obj._aggregate_select_cache = self._aggregate_select_cache.copy() + # _aggregate_select_cache cannot be copied, as doing so breaks the + # (necessary) state in which both aggregates and + # _aggregate_select_cache point to the same underlying objects. + # It will get re-populated in the cloned queryset the next time it's + # used. + obj._aggregate_select_cache = None obj.max_depth = self.max_depth obj.extra = self.extra.copy() if self.extra_select_mask is None: -- cgit v1.3