diff --git a/bin/dbus/cx.ring.Ring.ConfigurationManager.xml b/bin/dbus/cx.ring.Ring.ConfigurationManager.xml
index c47644337..f4c51d935 100644
--- a/bin/dbus/cx.ring.Ring.ConfigurationManager.xml
+++ b/bin/dbus/cx.ring.Ring.ConfigurationManager.xml
@@ -664,11 +664,11 @@
- Notify clients that a vcard has been received
+ Notify clients that a vCard has been received
-
+
diff --git a/bin/jni/configurationmanager.i b/bin/jni/configurationmanager.i
index c4c4ba207..580affb1f 100644
--- a/bin/jni/configurationmanager.i
+++ b/bin/jni/configurationmanager.i
@@ -35,7 +35,7 @@ public:
virtual void volatileAccountDetailsChanged(const std::string& account_id, const std::map& details){}
virtual void incomingAccountMessage(const std::string& /*account_id*/, const std::string& /*message_id*/, const std::string& /*from*/, const std::map& /*payload*/){}
virtual void accountMessageStatusChanged(const std::string& /*account_id*/, uint64_t /*message_id*/, const std::string& /*to*/, int /*state*/){}
- virtual void profileReceived(const std::string& /*account_id*/, const std::string& /*from*/, const std::string& /*vCard*/){}
+ virtual void profileReceived(const std::string& /*account_id*/, const std::string& /*from*/, const std::string& /*path*/){}
virtual void composingStatusChanged(const std::string& /*account_id*/, const std::string& /*from*/, int /*state*/){}
virtual void knownDevicesChanged(const std::string& /*account_id*/, const std::map& /*devices*/){}
virtual void exportOnRingEnded(const std::string& /*account_id*/, int /*state*/, const std::string& /*pin*/){}
@@ -244,7 +244,7 @@ public:
virtual void historyChanged(void){}
virtual void stunStatusFailure(const std::string& account_id){}
virtual void accountDetailsChanged(const std::string& account_id, const std::map& details){}
- virtual void profileReceived(const std::string& /*account_id*/, const std::string& /*from*/, const std::string& /*vCard*/) {}
+ virtual void profileReceived(const std::string& /*account_id*/, const std::string& /*from*/, const std::string& /*path*/) {}
virtual void registrationStateChanged(const std::string& account_id, const std::string& state, int code, const std::string& detail_str){}
virtual void volatileAccountDetailsChanged(const std::string& account_id, const std::map& details){}
virtual void incomingAccountMessage(const std::string& /*account_id*/, const std::string& /*message_id*/, const std::string& /*from*/, const std::map& /*payload*/){}
diff --git a/bin/nodejs/configurationmanager.i b/bin/nodejs/configurationmanager.i
index bac910e4a..6bd3da752 100644
--- a/bin/nodejs/configurationmanager.i
+++ b/bin/nodejs/configurationmanager.i
@@ -34,7 +34,7 @@ public:
virtual void volatileAccountDetailsChanged(const std::string& account_id, const std::map& details){}
virtual void incomingAccountMessage(const std::string& /*account_id*/, const std::string& /*from*/, const std::map& /*payload*/){}
virtual void accountMessageStatusChanged(const std::string& /*account_id*/, uint64_t /*message_id*/, const std::string& /*to*/, int /*state*/){}
- virtual void profileReceived(const std::string& /*account_id*/, const std::string& /*from*/, const std::string& /*vCard*/){}
+ virtual void profileReceived(const std::string& /*account_id*/, const std::string& /*from*/, const std::string& /*path*/){}
virtual void knownDevicesChanged(const std::string& /*account_id*/, const std::map& /*devices*/){}
virtual void exportOnRingEnded(const std::string& /*account_id*/, int /*state*/, const std::string& /*pin*/){}
@@ -234,7 +234,7 @@ public:
virtual void volatileAccountDetailsChanged(const std::string& account_id, const std::map& details){}
virtual void incomingAccountMessage(const std::string& /*account_id*/, const std::string& /*from*/, const std::map& /*payload*/){}
virtual void accountMessageStatusChanged(const std::string& /*account_id*/, uint64_t /*message_id*/, const std::string& /*to*/, int /*state*/){}
- virtual void profileReceived(const std::string& /*account_id*/, const std::string& /*from*/, const std::string& /*vCard*/){}
+ virtual void profileReceived(const std::string& /*account_id*/, const std::string& /*from*/, const std::string& /*path*/){}
virtual void knownDevicesChanged(const std::string& /*account_id*/, const std::map& /*devices*/){}
virtual void exportOnRingEnded(const std::string& /*account_id*/, int /*state*/, const std::string& /*pin*/){}
diff --git a/src/data_transfer.cpp b/src/data_transfer.cpp
index 4de252b40..4c3518665 100644
--- a/src/data_transfer.cpp
+++ b/src/data_transfer.cpp
@@ -122,12 +122,12 @@ protected:
void
DataTransfer::emit(DRing::DataTransferEventCode code) const
{
- if (internalCompletionCb_)
- return; // VCard transfer is just for the daemon
{
std::lock_guard lk {infoMutex_};
info_.lastEvent = code;
}
+ if (internalCompletionCb_)
+ return; // VCard transfer is just for the daemon
emitSignal(id, uint32_t(code));
}
@@ -643,11 +643,9 @@ IncomingFileTransfer::close() noexcept
JAMI_DBG() << "[FTP] file closed, rx " << info_.bytesProgress
<< " on " << info_.totalSize;
if (info_.bytesProgress >= info_.totalSize) {
- if (internalCompletionCb_) {
+ if (internalCompletionCb_)
internalCompletionCb_(info_.path);
- } else {
- emit(DRing::DataTransferEventCode::finished);
- }
+ emit(DRing::DataTransferEventCode::finished);
}
else
emit(DRing::DataTransferEventCode::closed_by_host);
diff --git a/src/jamidht/jamiaccount.cpp b/src/jamidht/jamiaccount.cpp
index abc71e49f..3c8984004 100644
--- a/src/jamidht/jamiaccount.cpp
+++ b/src/jamidht/jamiaccount.cpp
@@ -2055,12 +2055,12 @@ JamiAccount::doRegister_()
uint64_t tid;
std::istringstream iss(tid_str);
iss >> tid;
- dhtPeerConnector_->onIncomingConnection(peerId, tid, std::move(channel),
- [peerId, accountId=getAccountID()](const std::string& path) {
- auto vCard = fileutils::loadTextFile(path);
- emitSignal(accountId, peerId, vCard);
- fileutils::remove(path, true);
- });
+ std::function cb;
+ if (isVCard)
+ cb = [peerId, accountId=getAccountID()](const std::string& path) {
+ emitSignal(accountId, peerId, path);
+ };
+ dhtPeerConnector_->onIncomingConnection(peerId, tid, std::move(channel), std::move(cb));
}
}
});
@@ -3338,7 +3338,7 @@ JamiAccount::sendSIPMessage(SipConnection& conn, const std::string& to, void* ct
}
void
-JamiAccount::sendProfile(const std::string& peerId, const std::string& deviceId)
+JamiAccount::sendProfile(const std::string& deviceId)
{
try {
if (not needToSendProfile(deviceId)) {
@@ -3405,7 +3405,7 @@ JamiAccount::cacheSIPConnection(std::shared_ptr&& socket, const s
JAMI_WARN("New SIP channel opened with %s", deviceId.c_str());
lk.unlock();
- sendProfile(peerId, deviceId);
+ sendProfile(deviceId);
// Retry messages
messageEngine_.onPeerOnline(peerId);
diff --git a/src/jamidht/jamiaccount.h b/src/jamidht/jamiaccount.h
index 01eb612b9..6ab4cd127 100644
--- a/src/jamidht/jamiaccount.h
+++ b/src/jamidht/jamiaccount.h
@@ -747,10 +747,9 @@ private:
bool needToSendProfile(const std::string& deviceId);
/**
* Send Profile via cached SIP connection
- * @param peerId Peer related
* @param deviceId Device that will receive the profile
*/
- void sendProfile(const std::string& peerId, const std::string& deviceId);
+ void sendProfile(const std::string& deviceId);
};
static inline std::ostream& operator<< (std::ostream& os, const JamiAccount& acc)