mirrors/django
1
0
Fork 0
mirror of https://github.com/django/django.git synced 2025-07-04 09:49:12 +00:00
Code Issues 32 Releases Wiki Activity

magic-removal: Fixed #1624 -- Made small changes to parameters in executemany() in cursor implementations in MySQL and SQLite database backends

Browse Source 
git-svn-id: http://code.djangoproject.com/svn/django/branches/magic-removal@2686 bcc190cf-cafb-0310-a4f2-bffc1f526a37
This commit is contained in:
Adrian Holovaty 2006-04-12 13:19:45 +00:00
parent 6d953207ae
commit af0056201e
2 changed files with 5 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
django/db/backends/sqlite3/base.py
View File

@ -62,15 +62,14 @@ class SQLiteCursorWrapper(Database.Cursor):
"""
Django uses "format" style placeholders, but pysqlite2 uses "qmark" style.
This fixes it -- but note that if you want to use a literal "%s" in a query,
you'll need to use "%%s" (which I belive is true of other wrappers as well).
you'll need to use "%%s".
"""
def execute(self, query, params=[]):
def execute(self, query, params=()):
query = self.convert_query(query, len(params))
return Database.Cursor.execute(self, query, params)
def executemany(self, query, params=[]):
query = self.convert_query(query, len(params[0]))
def executemany(self, query, param_list):
query = self.convert_query(query, len(param_list[0]))
return Database.Cursor.executemany(self, query, params)
def convert_query(self, query, num_params):

2
django/db/backends/util.py
View File

@ -6,7 +6,7 @@ class CursorDebugWrapper:
self.cursor = cursor
self.db = db
def execute(self, sql, params=[]):
def execute(self, sql, params=()):
start = time()
try:
return self.cursor.execute(sql, params)