mod_iq: Optimized a bit (fewer table accesses).

Sun, 28 Nov 2010 02:37:18 +0500

author
Waqas Hussain <waqas20@gmail.com>
date
Sun, 28 Nov 2010 02:37:18 +0500
changeset 3658
07f0c2ef16cb
parent 3657
71fd1582f01b
child 3659
13600f0d56b5

mod_iq: Optimized a bit (fewer table accesses).

plugins/mod_iq.lua file | annotate | diff | comparison | revisions
--- a/plugins/mod_iq.lua	Sun Nov 28 02:24:26 2010 +0500
+++ b/plugins/mod_iq.lua	Sun Nov 28 02:37:18 2010 +0500
@@ -32,14 +32,15 @@
 module:hook("iq/bare", function(data)
 	-- IQ to bare JID recieved
 	local origin, stanza = data.origin, data.stanza;
+	local type = stanza.attr.type;
 
 	-- TODO fire post processing events
-	if stanza.attr.type == "get" or stanza.attr.type == "set" then
+	if type == "get" or type == "set" then
 		local ret = module:fire_event("iq/bare/"..stanza.tags[1].attr.xmlns..":"..stanza.tags[1].name, data);
 		if ret ~= nil then return ret; end
-		return module:fire_event("iq-"..stanza.attr.type.."/bare/"..stanza.tags[1].attr.xmlns..":"..stanza.tags[1].name, data);
+		return module:fire_event("iq-"..type.."/bare/"..stanza.tags[1].attr.xmlns..":"..stanza.tags[1].name, data);
 	else
-		module:fire_event("iq-"..stanza.attr.type.."/bare/"..stanza.attr.id, data);
+		module:fire_event("iq-"..type.."/bare/"..stanza.attr.id, data);
 		return true;
 	end
 end);
@@ -47,13 +48,14 @@
 module:hook("iq/self", function(data)
 	-- IQ to self JID recieved
 	local origin, stanza = data.origin, data.stanza;
+	local type = stanza.attr.type;
 
-	if stanza.attr.type == "get" or stanza.attr.type == "set" then
+	if type == "get" or type == "set" then
 		local ret = module:fire_event("iq/self/"..stanza.tags[1].attr.xmlns..":"..stanza.tags[1].name, data);
 		if ret ~= nil then return ret; end
-		return module:fire_event("iq-"..stanza.attr.type.."/self/"..stanza.tags[1].attr.xmlns..":"..stanza.tags[1].name, data);
+		return module:fire_event("iq-"..type.."/self/"..stanza.tags[1].attr.xmlns..":"..stanza.tags[1].name, data);
 	else
-		module:fire_event("iq-"..stanza.attr.type.."/self/"..stanza.attr.id, data);
+		module:fire_event("iq-"..type.."/self/"..stanza.attr.id, data);
 		return true;
 	end
 end);
@@ -61,13 +63,14 @@
 module:hook("iq/host", function(data)
 	-- IQ to a local host recieved
 	local origin, stanza = data.origin, data.stanza;
+	local type = stanza.attr.type;
 
-	if stanza.attr.type == "get" or stanza.attr.type == "set" then
+	if type == "get" or type == "set" then
 		local ret = module:fire_event("iq/host/"..stanza.tags[1].attr.xmlns..":"..stanza.tags[1].name, data);
 		if ret ~= nil then return ret; end
-		return module:fire_event("iq-"..stanza.attr.type.."/host/"..stanza.tags[1].attr.xmlns..":"..stanza.tags[1].name, data);
+		return module:fire_event("iq-"..type.."/host/"..stanza.tags[1].attr.xmlns..":"..stanza.tags[1].name, data);
 	else
-		module:fire_event("iq-"..stanza.attr.type.."/host/"..stanza.attr.id, data);
+		module:fire_event("iq-"..type.."/host/"..stanza.attr.id, data);
 		return true;
 	end
 end);

mercurial