1berry/qtbase
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

SQL/OCI: add maximumIdentifierLength()

Browse Source 
A table name or identifier must not be longer than 30 (< Oracle 12.2) or
128 bytes (sadly not characters).

Change-Id: I49192afaf908e12f5cfd20c754640b6117b03a71
Reviewed-by: Volker Hilsheimer <volker.hilsheimer@qt.io>
This commit is contained in:
Christian Ehrlicher 2023-03-12 14:50:05 +01:00
parent f49f3f2f94
commit 1981ebe8ad
3 changed files with 13 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
src/plugins/sqldrivers/oci/qsql_oci.cpp
View File

@ -2759,4 +2759,11 @@ QString QOCIDriver::escapeIdentifier(const QString &identifier, IdentifierType t
return res;
}
int QOCIDriver::maximumIdentifierLength(IdentifierType type) const
{
Q_D(const QOCIDriver);
Q_UNUSED(type);
return d->serverVersion > 12 ? 128 : 30;
}
QT_END_NAMESPACE

1
src/plugins/sqldrivers/oci/qsql_oci_p.h
View File

@ -58,6 +58,7 @@ public:
bool trimStrings) const override;
QVariant handle() const override;
QString escapeIdentifier(const QString &identifier, IdentifierType) const override;
int maximumIdentifierLength(IdentifierType type) const override;
protected:
bool beginTransaction() override;

13
tests/auto/sql/kernel/qsqldatabase/tst_databases.h
View File

@ -45,16 +45,13 @@ static QString qGetHostName()
return hostname;
}
inline QString fixupTableName(const QString &tableName, QSqlDatabase db)
inline QString fixupTableName(const QString &tableName, const QSqlDatabase &db)
{
QString tbName = tableName;
// On Oracle we are limited to 30 character tablenames
QSqlDriverPrivate *d = static_cast<QSqlDriverPrivate *>(QObjectPrivate::get(db.driver()));
if (d && d->dbmsType == QSqlDriver::Oracle)
tbName.truncate(30);
// On Interbase we are limited to 31 character tablenames
if (d && d->dbmsType == QSqlDriver::Interbase)
tbName.truncate(31);
// Oracle & Interbase/Firebird have a limit on the tablename length
QSqlDriver *drv = db.driver();
if (drv)
tbName.truncate(drv->maximumIdentifierLength(QSqlDriver::TableName));
return tbName;
}