diff --git a/src/blackmisc/network/authenticateduser.cpp b/src/blackmisc/network/authenticateduser.cpp index f4f697028..8b7b6bfbb 100644 --- a/src/blackmisc/network/authenticateduser.cpp +++ b/src/blackmisc/network/authenticateduser.cpp @@ -49,8 +49,8 @@ namespace BlackMisc QString CAuthenticatedUser::convertToQString(bool i18n) const { Q_UNUSED(i18n); - if (this->m_realname.isEmpty()) return ""; - QString s = this->m_realname; + if (m_realname.isEmpty()) return ""; + QString s = m_realname; if (this->hasValidDbKey()) { s.append(this->getDbKeyAsStringInParentheses(" ")); @@ -77,13 +77,13 @@ namespace BlackMisc void CAuthenticatedUser::setRealName(const QString &realname) { const QString rn(realname.trimmed().simplified()); - this->m_realname = rn; + m_realname = rn; } void CAuthenticatedUser::setUsername(const QString &username) { const QString un(username.trimmed().simplified().toUpper()); - this->m_username = un; + m_username = un; } CStatusMessageList CAuthenticatedUser::validate() const @@ -139,18 +139,12 @@ namespace BlackMisc const ColumnIndex i = index.frontCasted(); switch (i) { - case IndexVatsimId: - return CVariant::fromValue(this->m_vatsimId); - case IndexEmail: - return CVariant::fromValue(this->m_email); - case IndexPassword: - return CVariant::fromValue(this->m_password); - case IndexRealName: - return CVariant::fromValue(this->m_realname); - case IndexUsername: - return CVariant::fromValue(this->m_username); - default: - return CValueObject::propertyByIndex(index); + case IndexVatsimId: return CVariant::fromValue(m_vatsimId); + case IndexEmail: return CVariant::fromValue(m_email); + case IndexPassword: return CVariant::fromValue(m_password); + case IndexRealName: return CVariant::fromValue(m_realname); + case IndexUsername: return CVariant::fromValue(m_username); + default: return CValueObject::propertyByIndex(index); } } @@ -161,24 +155,12 @@ namespace BlackMisc const ColumnIndex i = index.frontCasted(); switch (i) { - case IndexVatsimId: - this->setVatsimId(variant.toInt()); - break; - case IndexEmail: - this->setEmail(variant.value()); - break; - case IndexPassword: - this->setPassword(variant.value()); - break; - case IndexRealName: - this->setRealName(variant.value()); - break; - case IndexUsername: - this->setUsername(variant.value()); - break; - default: - CValueObject::setPropertyByIndex(index, variant); - break; + case IndexVatsimId: this->setVatsimId(variant.toInt()); break; + case IndexEmail: this->setEmail(variant.value()); break; + case IndexPassword: this->setPassword(variant.value()); break; + case IndexRealName: this->setRealName(variant.value()); break; + case IndexUsername: this->setUsername(variant.value()); break; + default: CValueObject::setPropertyByIndex(index, variant); break; } } } // namespace diff --git a/src/blackmisc/network/authenticateduser.h b/src/blackmisc/network/authenticateduser.h index 83739726e..ff21a3e31 100644 --- a/src/blackmisc/network/authenticateduser.h +++ b/src/blackmisc/network/authenticateduser.h @@ -81,13 +81,13 @@ namespace BlackMisc void setPassword(const QString &pw) { m_password = pw.trimmed(); } //! Valid user object? - bool isValid() const { return !this->m_realname.isEmpty() && this->hasValidDbKey(); } + bool isValid() const { return !m_realname.isEmpty() && this->hasValidDbKey(); } //! Valid credentials? - bool hasValidCredentials() const { return this->isValid() && !this->m_password.isEmpty(); } + bool hasValidCredentials() const { return this->isValid() && !m_password.isEmpty(); } //! Valid real name? - bool hasValidRealName() const { return !this->m_realname.isEmpty(); } + bool hasValidRealName() const { return !m_realname.isEmpty(); } //! Validate, provide details about issues BlackMisc::CStatusMessageList validate() const; @@ -99,7 +99,7 @@ namespace BlackMisc void setEmail(const QString &email) { m_email = email.trimmed(); } //! Valid email? - bool hasValidEmail() const { return !this->m_email.isEmpty(); } + bool hasValidEmail() const { return !m_email.isEmpty(); } //! Get id. int getVatsimId() const { return m_vatsimId; } @@ -150,7 +150,7 @@ namespace BlackMisc void setEnabled(bool enabled) { m_enabled = enabled; } //! Enabled - bool isEnabled() const { return this->m_enabled; } + bool isEnabled() const { return m_enabled; } //! Entitled to directly update models bool canDirectlyWriteModels() const;