diff options
| author | Linus Jahn <lnj@kaidan.im> | 2022-09-05 20:37:05 +0200 |
|---|---|---|
| committer | Linus Jahn <lnj@kaidan.im> | 2022-09-06 00:46:56 +0200 |
| commit | 5f813f8a4bbe1e075daad7b3933829044befffb4 (patch) | |
| tree | 20102aa665b03cfb6dfe0c6ece94938320b6430e /src/client/QXmppRosterManager.cpp | |
| parent | 4c60b13e0b8011e4e0b093a690e939f3b23e2436 (diff) | |
| download | qxmpp-5f813f8a4bbe1e075daad7b3933829044befffb4.tar.gz | |
Reformat code
Diffstat (limited to 'src/client/QXmppRosterManager.cpp')
| -rw-r--r-- | src/client/QXmppRosterManager.cpp | 30 |
1 files changed, 20 insertions, 10 deletions
diff --git a/src/client/QXmppRosterManager.cpp b/src/client/QXmppRosterManager.cpp index 51a8b44a..bb4eb435 100644 --- a/src/client/QXmppRosterManager.cpp +++ b/src/client/QXmppRosterManager.cpp @@ -123,8 +123,9 @@ void QXmppRosterManager::_q_connected() roster.setMixAnnotate(true); d->rosterReqId = roster.id(); - if (client()->isAuthenticated()) + if (client()->isAuthenticated()) { client()->sendPacket(roster); + } } } @@ -139,21 +140,24 @@ void QXmppRosterManager::_q_disconnected() /// \cond bool QXmppRosterManager::handleStanza(const QDomElement &element) { - if (element.tagName() != "iq" || !QXmppRosterIq::isRosterIq(element)) + if (element.tagName() != "iq" || !QXmppRosterIq::isRosterIq(element)) { return false; + } // Security check: only server should send this iq // from() should be either empty or bareJid of the user const auto fromJid = element.attribute("from"); - if (!fromJid.isEmpty() && QXmppUtils::jidToBareJid(fromJid) != client()->configuration().jidBare()) + if (!fromJid.isEmpty() && QXmppUtils::jidToBareJid(fromJid) != client()->configuration().jidBare()) { return false; + } QXmppRosterIq rosterIq; rosterIq.parse(element); bool isInitial = (d->rosterReqId == rosterIq.id()); - if (isInitial) + if (isInitial) { d->rosterReqId.clear(); + } switch (rosterIq.type()) { case QXmppIq::Set: { @@ -210,8 +214,9 @@ void QXmppRosterManager::_q_presenceReceived(const QXmppPresence &presence) const auto bareJid = QXmppUtils::jidToBareJid(jid); const auto resource = QXmppUtils::jidToResource(jid); - if (bareJid.isEmpty()) + if (bareJid.isEmpty()) { return; + } switch (presence.type()) { case QXmppPresence::Available: @@ -424,15 +429,17 @@ bool QXmppRosterManager::removeItem(const QString &bareJid) /// bool QXmppRosterManager::renameItem(const QString &bareJid, const QString &name) { - if (!d->entries.contains(bareJid)) + if (!d->entries.contains(bareJid)) { return false; + } auto item = d->entries.value(bareJid); item.setName(name); // If there is a pending subscription, do not include the corresponding attribute in the stanza. - if (!item.subscriptionStatus().isEmpty()) + if (!item.subscriptionStatus().isEmpty()) { item.setSubscriptionStatus({}); + } QXmppRosterIq iq; iq.setType(QXmppIq::Set); @@ -490,8 +497,9 @@ QXmppRosterIq::Item QXmppRosterManager::getRosterEntry( const QString &bareJid) const { // will return blank entry if bareJid doesn't exist - if (d->entries.contains(bareJid)) + if (d->entries.contains(bareJid)) { return d->entries.value(bareJid); + } return {}; } @@ -503,8 +511,9 @@ QXmppRosterIq::Item QXmppRosterManager::getRosterEntry( /// QStringList QXmppRosterManager::getResources(const QString &bareJid) const { - if (d->presences.contains(bareJid)) + if (d->presences.contains(bareJid)) { return d->presences[bareJid].keys(); + } return {}; } @@ -519,8 +528,9 @@ QStringList QXmppRosterManager::getResources(const QString &bareJid) const QMap<QString, QXmppPresence> QXmppRosterManager::getAllPresencesForBareJid( const QString &bareJid) const { - if (d->presences.contains(bareJid)) + if (d->presences.contains(bareJid)) { return d->presences.value(bareJid); + } return {}; } |
