diff options
author | Friedemann Kleint <Friedemann.Kleint@qt.io> | 2021-04-12 11:39:21 +0200 |
---|---|---|
committer | Andy Shaw <andy.shaw@qt.io> | 2021-05-03 14:10:36 +0000 |
commit | 21b3b54193b9d90ee377fdfa306e21d2fdfe2837 (patch) | |
tree | 6ae3590c7840b33c67e23b2f54f79c087dfe8f68 /src/sql | |
parent | 18f5b83e97828dd572eaf67c054a43aa5472a668 (diff) |
QSqlTableModel::orderByClause(): Quote the table name
This ensures correct handling of names with special characters.
Pick-to: 5.15 6.1
Fixes: QTBUG-92584
Change-Id: I95c7c54d9c7ee00b221a55f3d07ef1ec3a3bd217
Reviewed-by: Andy Shaw <andy.shaw@qt.io>
Reviewed-by: Friedemann Kleint <Friedemann.Kleint@qt.io>
Reviewed-by: Edward Welbourne <edward.welbourne@qt.io>
Diffstat (limited to 'src/sql')
-rw-r--r-- | src/sql/models/qsqltablemodel.cpp | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/src/sql/models/qsqltablemodel.cpp b/src/sql/models/qsqltablemodel.cpp index 570101289c..5d1ce1784c 100644 --- a/src/sql/models/qsqltablemodel.cpp +++ b/src/sql/models/qsqltablemodel.cpp @@ -1001,7 +1001,8 @@ QString QSqlTableModel::orderByClause() const //we can safely escape the field because it would have been obtained from the database //and have the correct case - QString field = d->tableName + QLatin1Char('.') + QString field = d->db.driver()->escapeIdentifier(d->tableName, QSqlDriver::TableName) + + QLatin1Char('.') + d->db.driver()->escapeIdentifier(f.name(), QSqlDriver::FieldName); field = d->sortOrder == Qt::AscendingOrder ? Sql::asc(field) : Sql::desc(field); return Sql::orderBy(field); |