# HG changeset patch
# User Matthew Wild <mwild1@gmail.com>
# Date 1243296441 -3600
# Node ID 545c81364939ad3fa3a9fbc815b5c7f65b6c96f7
# Parent  1f4a4e6ae694b9272bcdf1fcbcd86a0fd76e855e
mod_roster: Don't send a query element in roster response if roster is unchanged

diff -r 1f4a4e6ae694 -r 545c81364939 plugins/mod_roster.lua
--- a/plugins/mod_roster.lua	Fri May 22 15:19:12 2009 +0100
+++ b/plugins/mod_roster.lua	Tue May 26 01:07:21 2009 +0100
@@ -35,12 +35,12 @@
 		function (session, stanza)
 			if stanza.tags[1].name == "query" then
 				if stanza.attr.type == "get" then
-					local roster = st.reply(stanza)
-								:query("jabber:iq:roster");
+					local roster = st.reply(stanza);
 					
 					local ver = stanza.tags[1].attr.ver
 					
 					if (not ver) or tonumber(ver) ~= (session.roster[false].version or 1) then
+						roster:query("jabber:iq:roster");
 						-- Client does not support versioning, or has stale roster
 						for jid in pairs(session.roster) do
 							if jid ~= "pending" and jid then