RawQuerySet.__repr__ fails when params passed as list

[Marti Raudsepp]

Django 1.2.3. The documentation suggests passing arguments to a Model.objects.raw() query as a list: ​http://docs.djangoproject.com/en/dev/topics/db/sql/#passing-parameters-into-raw

However, this breaks RawQuerySet.repr

In [1]: from django.contrib.auth import models

In [2]: r=models.User.objects.raw('select * from auth_user where id=%s', 10)

In [3]: repr(r)
Out[3]: "<RawQuerySet: 'select * from auth_user where id=10'>"

In [4]: r=models.User.objects.raw('select * from auth_user where id=%s and id=%s', (10, 10))

In [5]: repr(r)
Out[5]: "<RawQuerySet: 'select * from auth_user where id=10 and id=10'>"

In [6]: r=models.User.objects.raw('select * from auth_user where id=%s and id=%s', [10, 10])

In [7]: repr(r)
---------------------------------------------------------------------------
TypeError                                 Traceback (most recent call last)

/tmp/foo/<ipython console> in <module>()

/usr/lib/pymodules/python2.6/django/db/models/query.pyc in __repr__(self)
   1374 
   1375     def __repr__(self):
-> 1376         return "<RawQuerySet: %r>" % (self.raw_query % self.params)
   1377 
   1378     def __getitem__(self, k):

TypeError: not enough arguments for format string

[Alexey Smolsky]

As official python documentation says(​http://docs.python.org/library/stdtypes.html#string-formatting-operations), not-single argument for string formatting operation must be tuple or dictionary. So, we need pass list argument as tuple.

Patch with tests attached above.

[Alexey Smolsky]

Tests moved in other place

[Matthias Kestenholz]

Still applies correctly and fixes the reported problem.

Moving version back to 1.2 -- changing the version to SVN is not helpful because the field signifies since when a problem exists -- the fact that the ticket is still open already means that the problem still exists in SVN trunk.

Thanks!

[Jannis Leidel]

In [16088]:

Fixed #14729 -- RawQuerySet.repr fails when params passed as list. Thanks, intgr for ticket and accuser for patch.