summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorRemko Tronçon <git@el-tramo.be>2010-11-01 15:23:09 (GMT)
committerRemko Tronçon <git@el-tramo.be>2010-11-03 12:17:41 (GMT)
commit7e921ec72997493c5ab06cc13d7cf82bb7e8f643 (patch)
tree4f01be45f5bbb1f16862fc7e0dd2f3421d40ceeb /Swiften
parent210accbcb135a9551cb56e5197a3a5acf8d15f49 (diff)
downloadswift-7e921ec72997493c5ab06cc13d7cf82bb7e8f643.zip
swift-7e921ec72997493c5ab06cc13d7cf82bb7e8f643.tar.bz2
Give later IQ handlers precedence over older IQ handlers.
Diffstat (limited to 'Swiften')
-rw-r--r--Swiften/Queries/IQRouter.cpp5
-rw-r--r--Swiften/Queries/UnitTest/IQRouterTest.cpp8
2 files changed, 7 insertions, 6 deletions
diff --git a/Swiften/Queries/IQRouter.cpp b/Swiften/Queries/IQRouter.cpp
index a1689e9..7fa2dcf 100644
--- a/Swiften/Queries/IQRouter.cpp
+++ b/Swiften/Queries/IQRouter.cpp
@@ -33,8 +33,9 @@ void IQRouter::handleIQ(boost::shared_ptr<IQ> iq) {
queueRemoves_ = true;
bool handled = false;
- foreach(boost::shared_ptr<IQHandler> handler, handlers_) {
- handled |= handler->handleIQ(iq);
+ // Go through the handlers in reverse order, to give precedence to the last added handler
+ for (std::vector<boost::shared_ptr<IQHandler> >::const_reverse_iterator i = handlers_.rbegin(); i != handlers_.rend(); ++i) {
+ handled |= (*i)->handleIQ(iq);
if (handled) {
break;
}
diff --git a/Swiften/Queries/UnitTest/IQRouterTest.cpp b/Swiften/Queries/UnitTest/IQRouterTest.cpp
index 2d6b2cb..864108c 100644
--- a/Swiften/Queries/UnitTest/IQRouterTest.cpp
+++ b/Swiften/Queries/UnitTest/IQRouterTest.cpp
@@ -54,8 +54,8 @@ class IQRouterTest : public CppUnit::TestFixture
void testRemoveHandler_AfterHandleIQ() {
IQRouter testling(channel_);
- DummyIQHandler handler1(true, &testling);
DummyIQHandler handler2(true, &testling);
+ DummyIQHandler handler1(true, &testling);
channel_->onIQReceived(boost::shared_ptr<IQ>(new IQ()));
testling.removeHandler(&handler1);
@@ -67,8 +67,8 @@ class IQRouterTest : public CppUnit::TestFixture
void testHandleIQ_SuccesfulHandlerFirst() {
IQRouter testling(channel_);
- DummyIQHandler handler1(true, &testling);
DummyIQHandler handler2(false, &testling);
+ DummyIQHandler handler1(true, &testling);
channel_->onIQReceived(boost::shared_ptr<IQ>(new IQ()));
@@ -79,8 +79,8 @@ class IQRouterTest : public CppUnit::TestFixture
void testHandleIQ_SuccesfulHandlerLast() {
IQRouter testling(channel_);
- DummyIQHandler handler1(false, &testling);
DummyIQHandler handler2(true, &testling);
+ DummyIQHandler handler1(false, &testling);
channel_->onIQReceived(boost::shared_ptr<IQ>(new IQ()));
@@ -102,8 +102,8 @@ class IQRouterTest : public CppUnit::TestFixture
void testHandleIQ_HandlerRemovedDuringHandle() {
IQRouter testling(channel_);
- RemovingIQHandler handler1(&testling);
DummyIQHandler handler2(true, &testling);
+ RemovingIQHandler handler1(&testling);
channel_->onIQReceived(boost::shared_ptr<IQ>(new IQ()));
channel_->onIQReceived(boost::shared_ptr<IQ>(new IQ()));