Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with HTTPS or Subversion.

Download ZIP
Browse files

Fixed #1760 -- Unwound a subselect in an update for order_with_respec…

…t_to handling. Required for MySQL and doesn't hurt too much for other platforms. thanks, Christopher Lenz, James Turnbull and Simon Litchfield.

git-svn-id: http://code.djangoproject.com/svn/django/trunk@6187 bcc190cf-cafb-0310-a4f2-bffc1f526a37
  • Loading branch information...
commit 595bf6b691c8ee3738d84f5cf15670ec6c3a01a6 1 parent b7df720
@malcolmt malcolmt authored
Showing with 6 additions and 4 deletions.
  1. +6 −4 django/db/models/base.py
View
10 django/db/models/base.py
@@ -241,10 +241,12 @@ def save(self, raw=False):
placeholders = ['%s'] * len(field_names)
if self._meta.order_with_respect_to:
field_names.append(qn('_order'))
- # TODO: This assumes the database supports subqueries.
- placeholders.append('(SELECT COUNT(*) FROM %s WHERE %s = %%s)' % \
- (qn(self._meta.db_table), qn(self._meta.order_with_respect_to.column)))
- db_values.append(getattr(self, self._meta.order_with_respect_to.attname))
+ placeholders.append('%s')
+ subsel = 'SELECT COUNT(*) FROM %s WHERE %s = %%s' % (
+ qn(self._meta.db_table),
+ qn(self._meta.order_with_respect_to.column))
+ cursor.execute(subsel, (getattr(self, self._meta.order_with_respect_to.attname),))
+ db_values.append(cursor.fetchone()[0])
if db_values:
cursor.execute("INSERT INTO %s (%s) VALUES (%s)" % \
(qn(self._meta.db_table), ','.join(field_names),
Please sign in to comment.
Something went wrong with that request. Please try again.