diff --git a/src/VCardManager.cpp b/src/VCardManager.cpp
index dfc77b8..b316699 100644
--- a/src/VCardManager.cpp
+++ b/src/VCardManager.cpp
@@ -1,85 +1,86 @@
/*
* Kaidan - A user-friendly XMPP client for every device!
*
* Copyright (C) 2016-2020 Kaidan developers and contributors
* (see the LICENSE file for a full list of copyright authors)
*
* Kaidan is free software: you can redistribute it and/or modify
* it under the terms of the GNU General Public License as published by
* the Free Software Foundation, either version 3 of the License, or
* (at your option) any later version.
*
* In addition, as a special exception, the author of Kaidan gives
* permission to link the code of its release with the OpenSSL
* project's "OpenSSL" library (or with modified versions of it that
* use the same license as the "OpenSSL" library), and distribute the
* linked executables. You must obey the GNU General Public License in
* all respects for all of the code used other than "OpenSSL". If you
* modify this file, you may extend this exception to your version of
* the file, but you are not obligated to do so. If you do not wish to
* do so, delete this exception statement from your version.
*
* Kaidan is distributed in the hope that it will be useful,
* but WITHOUT ANY WARRANTY; without even the implied warranty of
* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
* GNU General Public License for more details.
*
* You should have received a copy of the GNU General Public License
* along with Kaidan. If not, see .
*/
#include "VCardManager.h"
#include "AvatarFileStorage.h"
#include "Kaidan.h"
#include
#include
#include
VCardManager::VCardManager(QXmppClient *client, AvatarFileStorage *avatars, QObject *parent)
: QObject(parent), client(client), manager(client->vCardManager()), avatarStorage(avatars)
{
connect(&manager, &QXmppVCardManager::vCardReceived, this, &VCardManager::handleVCard);
connect(client, &QXmppClient::presenceReceived, this, &VCardManager::handlePresence);
connect(Kaidan::instance(), &Kaidan::vCardRequested, this, &VCardManager::fetchVCard);
// Currently we're not requesting the own VCard on every connection because it is probably
// way too resource intensive on mobile connections with many reconnects.
// Actually we would need to request our own avatar, calculate the hash of it and publish
// that in our presence.
//
// XEP-0084: User Avatar - probably best option (as long as the servers support XEP-0398:
// User Avatar to vCard-Based Avatars Conversion)
}
void VCardManager::fetchVCard(const QString& jid)
{
if (client->state() == QXmppClient::ConnectedState)
client->vCardManager().requestVCard(jid);
else
qWarning() << "[VCardManager] Could not fetch VCard: Not connected to a server";
}
void VCardManager::handleVCard(const QXmppVCardIq &iq)
{
- if (!iq.photo().isEmpty())
- avatarStorage->addAvatar(QXmppUtils::jidToBareJid(iq.from()), iq.photo());
+ if (!iq.photo().isEmpty()) {
+ avatarStorage->addAvatar(QXmppUtils::jidToBareJid(iq.from().isEmpty() ? client->configuration().jid() : iq.from()), iq.photo());
+ }
emit vCardReceived(iq);
}
void VCardManager::handlePresence(const QXmppPresence &presence)
{
if (presence.vCardUpdateType() == QXmppPresence::VCardUpdateValidPhoto) {
QString hash = avatarStorage->getHashOfJid(QXmppUtils::jidToBareJid(presence.from()));
QString newHash = presence.photoHash().toHex();
// check if hash differs and we need to refetch the avatar
if (hash != newHash)
manager.requestVCard(QXmppUtils::jidToBareJid(presence.from()));
} else if (presence.vCardUpdateType() == QXmppPresence::VCardUpdateNoPhoto) {
QString bareJid = QXmppUtils::jidToBareJid(presence.from());
avatarStorage->clearAvatar(bareJid);
}
// ignore VCardUpdateNone (protocol unsupported) and VCardUpdateNotReady
}