diff options
author | Jocelyn Turcotte <jturcotte@woboq.com> | 2017-05-09 15:24:11 +0300 |
---|---|---|
committer | Jocelyn Turcotte <jturcotte@woboq.com> | 2017-05-11 18:22:59 +0300 |
commit | 4ad190a558c64aac846dc828438acdec5fe9c6ed (patch) | |
tree | 92879909a4036fd51f7ece53654a0474298409c6 /src/gui/accountstate.cpp | |
parent | 3a193655b3bdb9634c753e9e7b73a56469249655 (diff) |
Use Qt logging categories for logging
This gives more insight about the logs and allow setting fine-tuned
logging rules. The categories are set to only output Info by default
so this allows us to provide more concise logging while keeping the
ability to extract more information for a specific category when
developping or debugging customer issues.
Issue #5647
Diffstat (limited to 'src/gui/accountstate.cpp')
-rw-r--r-- | src/gui/accountstate.cpp | 13 |
1 files changed, 7 insertions, 6 deletions
diff --git a/src/gui/accountstate.cpp b/src/gui/accountstate.cpp index b92b0ae44..dae976134 100644 --- a/src/gui/accountstate.cpp +++ b/src/gui/accountstate.cpp @@ -19,12 +19,13 @@ #include "logger.h" #include "configfile.h" -#include <QDebug> #include <QSettings> #include <qfontmetrics.h> namespace OCC { +Q_LOGGING_CATEGORY(lcAccountState, "gui.account.state", QtInfoMsg) + AccountState::AccountState(AccountPtr account) : QObject() , _account(account) @@ -85,7 +86,7 @@ AccountState::State AccountState::state() const void AccountState::setState(State state) { if (_state != state) { - qDebug() << "AccountState state change: " + qCDebug(lcAccountState) << "AccountState state change: " << stateString(_state) << "->" << stateString(state); State oldState = _state; _state = state; @@ -170,7 +171,7 @@ void AccountState::checkConnectivity() } if (_connectionValidator) { - qDebug() << "ConnectionValidator already running, ignoring" << account()->displayName(); + qCDebug(lcAccountState) << "ConnectionValidator already running, ignoring" << account()->displayName(); return; } @@ -181,7 +182,7 @@ void AccountState::checkConnectivity() if (isConnected() && _timeSinceLastETagCheck.isValid() && _timeSinceLastETagCheck.elapsed() < polltime) { - //qDebug() << account()->displayName() << "The last ETag check succeeded within the last " << polltime/1000 << " secs. No connection check needed!"; + //qCDebug(lcAccountState) << account()->displayName() << "The last ETag check succeeded within the last " << polltime/1000 << " secs. No connection check needed!"; return; } @@ -216,12 +217,12 @@ void AccountState::checkConnectivity() void AccountState::slotConnectionValidatorResult(ConnectionValidator::Status status, const QStringList& errors) { if (isSignedOut()) { - qDebug() << "Signed out, ignoring" << connectionStatusString(status) << _account->url().toString(); + qCDebug(lcAccountState) << "Signed out, ignoring" << connectionStatusString(status) << _account->url().toString(); return; } if (_connectionStatus != status) { - qDebug() << "AccountState connection status change: " + qCDebug(lcAccountState) << "AccountState connection status change: " << connectionStatusString(_connectionStatus) << "->" << connectionStatusString(status); _connectionStatus = status; |