From 461ef575bcf778ba24b0be6b775098d4b80ae5e1 Mon Sep 17 00:00:00 2001 From: Andy Shaw Date: Tue, 16 Jan 2018 09:54:01 +0100 Subject: Always escape the table names when creating the SQL statement Since some databases are case sensitive if part of the query is quoted, then we should ensure that all instances of the table name are escaped unless the test is delibrately testing the non-escaped case. As a result, this commit also removes some expected failures pertaining to PostgreSQL and also adds an entry to the list of tables being dropped when a test is finished. [ChangeLog][Sql][PostgreSQL] QSqlDatabase is now stricter about table names when used with record() and primaryIndex(). If the tablename was not quoted when it was created, then the table name passed to record() and primaryIndex() needs to be in lower case so that PostgreSQL is able to find it. Fixes: QTBUG-65788 Change-Id: Id1f54cb66b761c39edf858501b730ede7eec1fd3 Reviewed-by: Paul Wicking Reviewed-by: Edward Welbourne --- src/sql/doc/snippets/code/doc_src_sql-driver.qdoc | 13 +++++++++++++ 1 file changed, 13 insertions(+) (limited to 'src/sql/doc/snippets/code') diff --git a/src/sql/doc/snippets/code/doc_src_sql-driver.qdoc b/src/sql/doc/snippets/code/doc_src_sql-driver.qdoc index d127bdf8a57..9709deeccbb 100644 --- a/src/sql/doc/snippets/code/doc_src_sql-driver.qdoc +++ b/src/sql/doc/snippets/code/doc_src_sql-driver.qdoc @@ -237,3 +237,16 @@ Could not create database object //! [38] QPSQLDriver::getResult: Query results lost - probably discarded on executing another SQL query. //! [38] + +//! [39] +CREATE TABLE "testTable" ("id" INTEGER); +//! [39] + +//! [40] +QString tableString("testTable"); +QSqlQuery q; +// Create table query is not quoted, therefore it is mapped to lower case +q.exec(QString("CREATE TABLE %1 (id INTEGER)").arg(tableString)); +// Call toLower() on the string so that it can be matched +QSqlRecord rec = database.record(tableString.toLower()); +//! [40] -- cgit v1.2.3