diff --git a/django/db/models/sql/query.py b/django/db/models/sql/query.py index b4525f1075..bf3b63f100 100644 --- a/django/db/models/sql/query.py +++ b/django/db/models/sql/query.py @@ -808,6 +808,7 @@ class Query(object): self.where.relabel_aliases(change_map) for pos, col in enumerate(self.select): if isinstance(col, (list, tuple)): + old_alias = col[0] self.select[pos] = (change_map.get(old_alias, old_alias), col[1]) else: col.relabel_aliases(change_map) diff --git a/tests/regressiontests/queries/models.py b/tests/regressiontests/queries/models.py index 49b45e83fd..2f027f8d36 100644 --- a/tests/regressiontests/queries/models.py +++ b/tests/regressiontests/queries/models.py @@ -966,6 +966,11 @@ about them and shouldn't do bad things. >>> expected == result True +Make sure bump_prefix() (an internal Query method) doesn't (re-)break. +>>> query = Tag.objects.values_list('id').order_by().query +>>> query.bump_prefix() +>>> print query.as_sql()[0] +SELECT U0."id" FROM "queries_tag" U0 """} # In Python 2.3 and the Python 2.6 beta releases, exceptions raised in __len__