mod_presence: Don't depend on user being online when calculating top resources.
authorWaqas Hussain <waqas20@gmail.com>
Thu, 11 Feb 2010 19:55:06 +0000 (00:55 +0500)
committerWaqas Hussain <waqas20@gmail.com>
Thu, 11 Feb 2010 19:55:06 +0000 (00:55 +0500)
plugins/mod_presence.lua

index 4aa8f497f52523737edfdd570e2e3ab2846555bd..f6ea9e6b7b36cd91298b651e42ae07bfb43fff9b 100644 (file)
@@ -55,10 +55,11 @@ local function select_top_resources(user)
        end
        return recipients;
 end
-local function recalc_resource_map(origin)
-       local user = hosts[origin.host].sessions[origin.username];
-       user.top_resources = select_top_resources(user);
-       if #user.top_resources == 0 then user.top_resources = nil; end
+local function recalc_resource_map(user)
+       if user then
+               user.top_resources = select_top_resources(user);
+               if #user.top_resources == 0 then user.top_resources = nil; end
+       end
 end
 
 function handle_normal_presence(origin, stanza, core_route_stanza)
@@ -117,7 +118,7 @@ function handle_normal_presence(origin, stanza, core_route_stanza)
                origin.presence = nil;
                if origin.priority then
                        origin.priority = nil;
-                       recalc_resource_map(origin);
+                       recalc_resource_map(user);
                end
                if origin.directed then
                        for jid in pairs(origin.directed) do
@@ -139,7 +140,7 @@ function handle_normal_presence(origin, stanza, core_route_stanza)
                else priority = 0; end
                if origin.priority ~= priority then
                        origin.priority = priority;
-                       recalc_resource_map(origin);
+                       recalc_resource_map(user);
                end
        end
        stanza.attr.to = nil; -- reset it