Swift
summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorTobias Markmann <tm@ayena.de>2015-09-25 14:06:35 (GMT)
committerTobias Markmann <tm@ayena.de>2015-09-25 14:06:35 (GMT)
commit37aafcb4d693a0b4f5944a52e0c070e5aa384245 (patch)
treef6fe68c584de96992c2d2fde1f3cc7c3d702e084
parent2609a8a63662dfea9dccce2fd135b11a8d6c7875 (diff)
downloadswift-37aafcb4d693a0b4f5944a52e0c070e5aa384245.zip
swift-37aafcb4d693a0b4f5944a52e0c070e5aa384245.tar.bz2
Do not flush vCard cache if IQ request returns an error
Test-Information: Adjusted the corresponding test case accordingly. Change-Id: I4626f09cd51bac2ff9ca97d5a7da5a173fc627a5
Diffstat
-rw-r--r--Swiften/VCards/UnitTest/VCardManagerTest.cpp25
-rw-r--r--Swiften/VCards/VCardManager.cpp17
2 files changed, 22 insertions, 20 deletions
diff --git a/Swiften/VCards/UnitTest/VCardManagerTest.cpp b/Swiften/VCards/UnitTest/VCardManagerTest.cpp
index 32c91cf..278449b 100644
--- a/Swiften/VCards/UnitTest/VCardManagerTest.cpp
+++ b/Swiften/VCards/UnitTest/VCardManagerTest.cpp
@@ -1,23 +1,24 @@
1/* 1/*
2 * Copyright (c) 2010-2013 Isode Limited. 2 * Copyright (c) 2010-2015 Isode Limited.
3 * All rights reserved. 3 * All rights reserved.
4 * See the COPYING file for more information. 4 * See the COPYING file for more information.
5 */ 5 */
6 6
7#include <Swiften/Base/ByteArray.h>
8
9#include <cppunit/extensions/HelperMacros.h>
10#include <cppunit/extensions/TestFactoryRegistry.h>
11#include <vector> 7#include <vector>
8
12#include <boost/bind.hpp> 9#include <boost/bind.hpp>
13#include <boost/smart_ptr/make_shared.hpp> 10#include <boost/smart_ptr/make_shared.hpp>
14 11
15#include <Swiften/VCards/VCardManager.h> 12#include <cppunit/extensions/HelperMacros.h>
16#include <Swiften/VCards/VCardMemoryStorage.h> 13#include <cppunit/extensions/TestFactoryRegistry.h>
17#include <Swiften/Queries/IQRouter.h> 14
15#include <Swiften/Base/ByteArray.h>
18#include <Swiften/Client/DummyStanzaChannel.h> 16#include <Swiften/Client/DummyStanzaChannel.h>
19#include <Swiften/Crypto/CryptoProvider.h> 17#include <Swiften/Crypto/CryptoProvider.h>
20#include <Swiften/Crypto/PlatformCryptoProvider.h> 18#include <Swiften/Crypto/PlatformCryptoProvider.h>
19#include <Swiften/Queries/IQRouter.h>
20#include <Swiften/VCards/VCardManager.h>
21#include <Swiften/VCards/VCardMemoryStorage.h>
21 22
22using namespace Swift; 23using namespace Swift;
23 24
@@ -37,6 +38,8 @@ class VCardManagerTest : public CppUnit::TestFixture {
37 38
38 public: 39 public:
39 void setUp() { 40 void setUp() {
41 changes.clear();
42 ownChanges.clear();
40 ownJID = JID("baz@fum.com/dum"); 43 ownJID = JID("baz@fum.com/dum");
41 crypto = boost::shared_ptr<CryptoProvider>(PlatformCryptoProvider::create()); 44 crypto = boost::shared_ptr<CryptoProvider>(PlatformCryptoProvider::create());
42 stanzaChannel = new DummyStanzaChannel(); 45 stanzaChannel = new DummyStanzaChannel();
@@ -97,10 +100,8 @@ class VCardManagerTest : public CppUnit::TestFixture {
97 testling->requestVCard(JID("foo@bar.com/baz")); 100 testling->requestVCard(JID("foo@bar.com/baz"));
98 stanzaChannel->onIQReceived(IQ::createError(JID("baz@fum.com/foo"), stanzaChannel->sentStanzas[0]->getTo(), stanzaChannel->sentStanzas[0]->getID())); 101 stanzaChannel->onIQReceived(IQ::createError(JID("baz@fum.com/foo"), stanzaChannel->sentStanzas[0]->getTo(), stanzaChannel->sentStanzas[0]->getID()));
99 102
100 CPPUNIT_ASSERT_EQUAL(1, static_cast<int>(changes.size())); 103 // On error, cached vCards should not be changed.
101 CPPUNIT_ASSERT_EQUAL(JID("foo@bar.com/baz"), changes[0].first); 104 CPPUNIT_ASSERT_EQUAL(0, static_cast<int>(changes.size()));
102 CPPUNIT_ASSERT_EQUAL(std::string(""), changes[0].second->getFullName());
103 CPPUNIT_ASSERT_EQUAL(std::string(""), vcardStorage->getVCard(JID("foo@bar.com/baz"))->getFullName());
104 } 105 }
105 106
106 void testRequest_VCardAlreadyRequested() { 107 void testRequest_VCardAlreadyRequested() {
diff --git a/Swiften/VCards/VCardManager.cpp b/Swiften/VCards/VCardManager.cpp
index 5e552d6..5fa44ae 100644
--- a/Swiften/VCards/VCardManager.cpp
+++ b/Swiften/VCards/VCardManager.cpp
@@ -1,5 +1,5 @@
1/* 1/*
2 * Copyright (c) 2010-2014 Isode Limited. 2 * Copyright (c) 2010-2015 Isode Limited.
3 * All rights reserved. 3 * All rights reserved.
4 * See the COPYING file for more information. 4 * See the COPYING file for more information.
5 */ 5 */
@@ -10,9 +10,8 @@
10 10
11#include <Swiften/Base/Log.h> 11#include <Swiften/Base/Log.h>
12#include <Swiften/JID/JID.h> 12#include <Swiften/JID/JID.h>
13#include <Swiften/VCards/VCardStorage.h>
14#include <Swiften/VCards/GetVCardRequest.h> 13#include <Swiften/VCards/GetVCardRequest.h>
15 14#include <Swiften/VCards/VCardStorage.h>
16 15
17namespace Swift { 16namespace Swift {
18 17
@@ -53,12 +52,14 @@ void VCardManager::requestOwnVCard() {
53 52
54 53
55void VCardManager::handleVCardReceived(const JID& actualJID, VCard::ref vcard, ErrorPayload::ref error) { 54void VCardManager::handleVCardReceived(const JID& actualJID, VCard::ref vcard, ErrorPayload::ref error) {
56 if (error || !vcard) {
57 vcard = VCard::ref(new VCard());
58 }
59 requestedVCards.erase(actualJID); 55 requestedVCards.erase(actualJID);
60 JID jid = actualJID.isValid() ? actualJID : ownJID.toBare(); 56 if (!error) {
61 setVCard(jid, vcard); 57 if (!vcard) {
58 vcard = VCard::ref(new VCard());
59 }
60 JID jid = actualJID.isValid() ? actualJID : ownJID.toBare();
61 setVCard(jid, vcard);
62 }
62} 63}
63 64
64SetVCardRequest::ref VCardManager::createSetVCardRequest(VCard::ref vcard) { 65SetVCardRequest::ref VCardManager::createSetVCardRequest(VCard::ref vcard) {