SQL/QSqlQuery: use categorized logger

Use the categorized logger qt.sql.qsqlquery

Change-Id: Ib5c26751ef013261080207fc13b80b91c644a640
Reviewed-by: Axel Spoerl <axel.spoerl@qt.io>
This commit is contained in:
Christian Ehrlicher 2024-04-17 20:33:11 +02:00
parent b773a4f583
commit 6bc454b247

View File

@ -7,6 +7,7 @@
#include "qatomic.h" #include "qatomic.h"
#include "qdebug.h" #include "qdebug.h"
#include "qloggingcategory.h"
#include "qsqlrecord.h" #include "qsqlrecord.h"
#include "qsqlresult.h" #include "qsqlresult.h"
#include "qsqldriver.h" #include "qsqldriver.h"
@ -19,6 +20,8 @@
QT_BEGIN_NAMESPACE QT_BEGIN_NAMESPACE
static Q_LOGGING_CATEGORY(lcSqlQuery, "qt.sql.qsqlquery")
class QSqlQueryPrivate class QSqlQueryPrivate
{ {
public: public:
@ -346,7 +349,7 @@ bool QSqlQuery::isNull(QStringView name) const
qsizetype index = d->sqlResult->record().indexOf(name); qsizetype index = d->sqlResult->record().indexOf(name);
if (index > -1) if (index > -1)
return isNull(index); return isNull(index);
qWarning("QSqlQuery::isNull: unknown field name '%s'", qPrintable(name.toString())); qCWarning(lcSqlQuery, "QSqlQuery::isNull: unknown field name '%ls'", qUtf16Printable(name.toString()));
return true; return true;
} }
@ -382,7 +385,7 @@ bool QSqlQuery::exec(const QString& query)
t.start(); t.start();
#endif #endif
if (!driver()) { if (!driver()) {
qWarning("QSqlQuery::exec: called before driver has been set up"); qCWarning(lcSqlQuery, "QSqlQuery::exec: called before driver has been set up");
return false; return false;
} }
if (d->ref.loadRelaxed() != 1) { if (d->ref.loadRelaxed() != 1) {
@ -399,19 +402,20 @@ bool QSqlQuery::exec(const QString& query)
} }
d->sqlResult->setQuery(query.trimmed()); d->sqlResult->setQuery(query.trimmed());
if (!driver()->isOpen() || driver()->isOpenError()) { if (!driver()->isOpen() || driver()->isOpenError()) {
qWarning("QSqlQuery::exec: database not open"); qCWarning(lcSqlQuery, "QSqlQuery::exec: database not open");
return false; return false;
} }
if (query.isEmpty()) { if (query.isEmpty()) {
qWarning("QSqlQuery::exec: empty query"); qCWarning(lcSqlQuery, "QSqlQuery::exec: empty query");
return false; return false;
} }
bool retval = d->sqlResult->reset(query); bool retval = d->sqlResult->reset(query);
#ifdef QT_DEBUG_SQL #ifdef QT_DEBUG_SQL
qDebug().nospace() << "Executed query (" << t.elapsed() << "ms, " << d->sqlResult->size() qCDebug(lcSqlQuery()).nospace() << "Executed query (" << t.elapsed() << "ms, "
<< " results, " << d->sqlResult->numRowsAffected() << d->sqlResult->size()
<< " affected): " << d->sqlResult->lastQuery(); << " results, " << d->sqlResult->numRowsAffected()
<< " affected): " << d->sqlResult->lastQuery();
#endif #endif
return retval; return retval;
} }
@ -439,7 +443,7 @@ QVariant QSqlQuery::value(int index) const
{ {
if (isActive() && isValid() && (index > -1)) if (isActive() && isValid() && (index > -1))
return d->sqlResult->data(index); return d->sqlResult->data(index);
qWarning("QSqlQuery::value: not positioned on a valid record"); qCWarning(lcSqlQuery, "QSqlQuery::value: not positioned on a valid record");
return QVariant(); return QVariant();
} }
@ -464,7 +468,7 @@ QVariant QSqlQuery::value(QStringView name) const
qsizetype index = d->sqlResult->record().indexOf(name); qsizetype index = d->sqlResult->record().indexOf(name);
if (index > -1) if (index > -1)
return value(index); return value(index);
qWarning("QSqlQuery::value: unknown field name '%s'", qPrintable(name.toString())); qCWarning(lcSqlQuery, "QSqlQuery::value: unknown field name '%ls'", qUtf16Printable(name.toString()));
return QVariant(); return QVariant();
} }
@ -610,7 +614,7 @@ bool QSqlQuery::seek(int index, bool relative)
} }
// let drivers optimize // let drivers optimize
if (isForwardOnly() && actualIdx < at()) { if (isForwardOnly() && actualIdx < at()) {
qWarning("QSqlQuery::seek: cannot seek backwards in a forward only query"); qCWarning(lcSqlQuery, "QSqlQuery::seek: cannot seek backwards in a forward only query");
return false; return false;
} }
if (actualIdx == (at() + 1) && at() != QSql::BeforeFirstRow) { if (actualIdx == (at() + 1) && at() != QSql::BeforeFirstRow) {
@ -716,7 +720,7 @@ bool QSqlQuery::previous()
if (!isSelect() || !isActive()) if (!isSelect() || !isActive())
return false; return false;
if (isForwardOnly()) { if (isForwardOnly()) {
qWarning("QSqlQuery::seek: cannot seek backwards in a forward only query"); qCWarning(lcSqlQuery, "QSqlQuery::seek: cannot seek backwards in a forward only query");
return false; return false;
} }
@ -749,7 +753,7 @@ bool QSqlQuery::first()
if (!isSelect() || !isActive()) if (!isSelect() || !isActive())
return false; return false;
if (isForwardOnly() && at() > QSql::BeforeFirstRow) { if (isForwardOnly() && at() > QSql::BeforeFirstRow) {
qWarning("QSqlQuery::seek: cannot seek backwards in a forward only query"); qCWarning(lcSqlQuery, "QSqlQuery::seek: cannot seek backwards in a forward only query");
return false; return false;
} }
return d->sqlResult->fetchFirst(); return d->sqlResult->fetchFirst();
@ -996,19 +1000,19 @@ bool QSqlQuery::prepare(const QString& query)
d->sqlResult->setNumericalPrecisionPolicy(d->sqlResult->numericalPrecisionPolicy()); d->sqlResult->setNumericalPrecisionPolicy(d->sqlResult->numericalPrecisionPolicy());
} }
if (!driver()) { if (!driver()) {
qWarning("QSqlQuery::prepare: no driver"); qCWarning(lcSqlQuery, "QSqlQuery::prepare: no driver");
return false; return false;
} }
if (!driver()->isOpen() || driver()->isOpenError()) { if (!driver()->isOpen() || driver()->isOpenError()) {
qWarning("QSqlQuery::prepare: database not open"); qCWarning(lcSqlQuery, "QSqlQuery::prepare: database not open");
return false; return false;
} }
if (query.isEmpty()) { if (query.isEmpty()) {
qWarning("QSqlQuery::prepare: empty query"); qCWarning(lcSqlQuery, "QSqlQuery::prepare: empty query");
return false; return false;
} }
#ifdef QT_DEBUG_SQL #ifdef QT_DEBUG_SQL
qDebug("\n QSqlQuery::prepare: %s", query.toLocal8Bit().constData()); qCDebug(lcSqlQuery, "\n QSqlQuery::prepare: %ls", qUtf16Printable(query));
#endif #endif
return d->sqlResult->savePrepare(query); return d->sqlResult->savePrepare(query);
} }
@ -1035,9 +1039,9 @@ bool QSqlQuery::exec()
bool retval = d->sqlResult->exec(); bool retval = d->sqlResult->exec();
#ifdef QT_DEBUG_SQL #ifdef QT_DEBUG_SQL
qDebug().nospace() << "Executed prepared query (" << t.elapsed() << "ms, " qCDebug(lcSqlQuery).nospace() << "Executed prepared query (" << t.elapsed() << "ms, "
<< d->sqlResult->size() << " results, " << d->sqlResult->numRowsAffected() << d->sqlResult->size() << " results, " << d->sqlResult->numRowsAffected()
<< " affected): " << d->sqlResult->lastQuery(); << " affected): " << d->sqlResult->lastQuery();
#endif #endif
return retval; return retval;
} }