diff options
| author | R David Murray <rdmurray@bitdance.com> | 2013-01-10 16:04:09 (GMT) |
|---|---|---|
| committer | R David Murray <rdmurray@bitdance.com> | 2013-01-10 16:04:09 (GMT) |
| commit | b52312923bee35b86fc072ec546cc3e588ae64c9 (patch) | |
| tree | 621c1002f13493da15de8719e084ac2d0775cf11 /Lib/sqlite3/dump.py | |
| parent | 78470b4c3a423fb304438be269afbbdd2247d676 (diff) | |
| download | cpython-b52312923bee35b86fc072ec546cc3e588ae64c9.zip cpython-b52312923bee35b86fc072ec546cc3e588ae64c9.tar.gz cpython-b52312923bee35b86fc072ec546cc3e588ae64c9.tar.bz2 | |
#15545: fix sqlite3.iterdump regression on unsortable row_factory objects.
The fix for issue 9750 introduced a regression by sorting the row objects
returned by fetchall. But if a row_factory such as sqlite3.Row is used, the
rows may not be sortable (in Python3), which leads to an exception. The
sorting is still a nice idea, so the patch moves the sort into the sql.
Fix and test by Peter Otten.
Diffstat (limited to 'Lib/sqlite3/dump.py')
| -rw-r--r-- | Lib/sqlite3/dump.py | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/Lib/sqlite3/dump.py b/Lib/sqlite3/dump.py index da6be68..de9c368 100644 --- a/Lib/sqlite3/dump.py +++ b/Lib/sqlite3/dump.py @@ -25,9 +25,10 @@ def _iterdump(connection): FROM "sqlite_master" WHERE "sql" NOT NULL AND "type" == 'table' + ORDER BY "name" """ schema_res = cu.execute(q) - for table_name, type, sql in sorted(schema_res.fetchall()): + for table_name, type, sql in schema_res.fetchall(): if table_name == 'sqlite_sequence': yield('DELETE FROM "sqlite_sequence";') elif table_name == 'sqlite_stat1': |