modulemanager: Fixed: Stanza modules were being auto-loaded for components (regressio...
[prosody.git] / core / modulemanager.lua
index 57a44c29df5bd4ce58ef9849711c3ee2ef2dee8b..f703889f4d512a3ef67f309a8455389e930c00b8 100644 (file)
@@ -1,4 +1,4 @@
--- Prosody IM v0.4
+-- Prosody IM
 -- Copyright (C) 2008-2009 Matthew Wild
 -- Copyright (C) 2008-2009 Waqas Hussain
 -- 
@@ -6,20 +6,19 @@
 -- COPYING file in the source package for more information.
 --
 
-
-
 local plugin_dir = CFG_PLUGINDIR or "./plugins/";
 
 local logger = require "util.logger";
 local log = logger.init("modulemanager");
-local addDiscoInfoHandler = require "core.discomanager".addDiscoInfoHandler;
 local eventmanager = require "core.eventmanager";
 local config = require "core.configmanager";
 local multitable_new = require "util.multitable".new;
 local register_actions = require "core.actions".register;
 local st = require "util.stanza";
+local pluginloader = require "util.pluginloader";
 
 local hosts = hosts;
+local prosody = prosody;
 
 local loadfile, pcall = loadfile, pcall;
 local setmetatable, setfenv, getfenv = setmetatable, setfenv, getfenv;
@@ -28,15 +27,18 @@ local t_insert, t_concat = table.insert, table.concat;
 local type = type;
 local next = next;
 local rawget = rawget;
-
+local error = error;
 local tostring = tostring;
 
+local autoload_modules = {"presence", "message", "iq"};
+
 -- We need this to let modules access the real global namespace
 local _G = _G;
 
 module "modulemanager"
 
-local api = {}; -- Module API container
+api = {};
+local api = api; -- Module API container
 
 local modulemap = { ["*"] = {} };
 
@@ -45,39 +47,47 @@ local handler_info = {};
 
 local modulehelpers = setmetatable({}, { __index = _G });
 
-local features_table = multitable_new();
 local handler_table = multitable_new();
 local hooked = multitable_new();
+local hooks = multitable_new();
 local event_hooks = multitable_new();
 
 local NULL = {};
 
 -- Load modules when a host is activated
 function load_modules_for_host(host)
-       if config.get(host, "core", "modules_enable") == false then
-               return; -- Only load for hosts, not components, etc.
-       end
-
-       -- Load modules from global section
-       local modules_enabled = config.get("*", "core", "modules_enabled");
-       local modules_disabled = config.get(host, "core", "modules_disabled");
        local disabled_set = {};
-       if modules_enabled then
-               if modules_disabled then
-                       for _, module in ipairs(modules_disabled) do
-                               disabled_set[module] = true;
-                       end
+       local modules_disabled = config.get(host, "core", "modules_disabled");
+       if modules_disabled then
+               for _, module in ipairs(modules_disabled) do
+                       disabled_set[module] = true;
                end
-               for _, module in ipairs(modules_enabled) do
+       end
+
+       -- Load auto-loaded modules for this host
+       if hosts[host].type == "local" then
+               for _, module in ipairs(autoload_modules) do
                        if not disabled_set[module] then
                                load(host, module);
                        end
                end
        end
 
+       -- Load modules from global section
+       if config.get(host, "core", "load_global_modules") ~= false then
+               local modules_enabled = config.get("*", "core", "modules_enabled");
+               if modules_enabled then
+                       for _, module in ipairs(modules_enabled) do
+                               if not disabled_set[module] and not is_loaded(host, module) then
+                                       load(host, module);
+                               end
+                       end
+               end
+       end
+       
        -- Load modules from just this host
        local modules_enabled = config.get(host, "core", "modules_enabled");
-       if modules_enabled then
+       if modules_enabled and modules_enabled ~= config.get("*", "core", "modules_enabled") then
                for _, module in pairs(modules_enabled) do
                        if not is_loaded(host, module) then
                                load(host, module);
@@ -86,6 +96,7 @@ function load_modules_for_host(host)
        end
 end
 eventmanager.add_event_hook("host-activated", load_modules_for_host);
+eventmanager.add_event_hook("component-activated", load_modules_for_host);
 --
 
 function load(host, module_name, config)
@@ -105,7 +116,7 @@ function load(host, module_name, config)
        end
        
 
-       local mod, err = loadfile(get_module_filename(module_name));
+       local mod, err = pluginloader.load_code(module_name);
        if not mod then
                log("error", "Unable to load module '%s': %s", module_name or "nil", err or "nil");
                return nil, err;
@@ -118,13 +129,21 @@ function load(host, module_name, config)
        
        setfenv(mod, pluginenv);
        if not hosts[host] then hosts[host] = { type = "component", host = host, connected = false, s2sout = {} }; end
+       hosts[host].modules = modulemap[host];
        
        local success, ret = pcall(mod);
        if not success then
-               log("error", "Error initialising module '%s': %s", name or "nil", ret or "nil");
+               log("error", "Error initialising module '%s': %s", module_name or "nil", ret or "nil");
                return nil, ret;
        end
        
+       if module_has_method(pluginenv, "load") then
+               local ok, err = call_module_method(pluginenv, "load");
+               if (not ok) and err then
+                       log("warn", "Error loading module '%s' on '%s': %s", module_name, host, err);
+               end
+       end
+
        -- Use modified host, if the module set one
        modulemap[api_instance.host][module_name] = pluginenv;
        
@@ -153,8 +172,6 @@ function unload(host, name, ...)
                        log("warn", "Non-fatal error unloading module '%s' on '%s': %s", name, host, err);
                end
        end
-       modulemap[host][name] = nil;
-       features_table:remove(host, name);
        local params = handler_table:get(host, name); -- , {module.host, origin_type, tag, xmlns}
        for _, param in pairs(params or NULL) do
                local handlers = stanza_handlers:get(param[1], param[2], param[3], param[4]);
@@ -164,6 +181,14 @@ function unload(host, name, ...)
                end
        end
        event_hooks:remove(host, name);
+       -- unhook event handlers hooked by module:hook
+       for event, handlers in pairs(hooks:get(host, name) or NULL) do
+               for handler in pairs(handlers or NULL) do
+                       (hosts[host] or prosody).events.remove_handler(event, handler);
+               end
+       end
+       hooks:remove(host, name);
+       modulemap[host][name] = nil;
        return true;
 end
 
@@ -171,9 +196,9 @@ function reload(host, name, ...)
        local mod = get_module(host, name);
        if not mod then return nil, "module-not-loaded"; end
 
-       local _mod, err = loadfile(get_module_filename(name)); -- checking for syntax errors
+       local _mod, err = pluginloader.load_code(name); -- checking for syntax errors
        if not _mod then
-               log("error", "Unable to load module '%s': %s", module_name or "nil", err or "nil");
+               log("error", "Unable to load module '%s': %s", name or "nil", err or "nil");
                return nil, err;
        end
 
@@ -210,7 +235,7 @@ function reload(host, name, ...)
 end
 
 function handle_stanza(host, origin, stanza)
-       local name, xmlns, origin_type = stanza.name, stanza.attr.xmlns, origin.type;
+       local name, xmlns, origin_type = stanza.name, stanza.attr.xmlns or "jabber:client", origin.type;
        if name == "iq" and xmlns == "jabber:client" then
                if stanza.attr.type == "get" or stanza.attr.type == "set" then
                        xmlns = stanza.tags[1].attr.xmlns or "jabber:client";
@@ -227,12 +252,13 @@ function handle_stanza(host, origin, stanza)
                (handlers[1])(origin, stanza);
                return true;
        else
-               log("debug", "Unhandled %s stanza: %s; xmlns=%s", origin.type, stanza.name, xmlns); -- we didn't handle it
                if stanza.attr.xmlns == "jabber:client" then
+                       log("debug", "Unhandled %s stanza: %s; xmlns=%s", origin.type, stanza.name, xmlns); -- we didn't handle it
                        if stanza.attr.type ~= "error" and stanza.attr.type ~= "result" then
                                origin.send(st.error_reply(stanza, "cancel", "service-unavailable"));
                        end
-               elseif not(name == "features" and xmlns == "http://etherx.jabber.org/streams") then -- FIXME remove check once we handle S2S features
+               elseif not((name == "features" or name == "error") and xmlns == "http://etherx.jabber.org/streams") then -- FIXME remove check once we handle S2S features
+                       log("warn", "Unhandled %s stream element: %s; xmlns=%s: %s", origin.type, stanza.name, xmlns, tostring(stanza)); -- we didn't handle it
                        origin:close("unsupported-stanza-type");
                end
        end
@@ -251,12 +277,6 @@ function call_module_method(module, method, ...)
        end
 end
 
-local _modulepath = { plugin_dir, "mod_", nil, ".lua"};
-function get_module_filename(name)
-       _modulepath[3] = name;
-       return t_concat(_modulepath);
-end
-
 ----- API functions exposed to modules -----------
 -- Must all be in api.* 
 
@@ -309,31 +329,11 @@ function api:add_iq_handler(origin_type, xmlns, handler)
        self:add_handler(origin_type, "iq", xmlns, handler);
 end
 
-addDiscoInfoHandler("*host", function(reply, to, from, node)
-       if #node == 0 then
-               local done = {};
-               for module, features in pairs(features_table:get(to) or NULL) do -- for each module
-                       for feature in pairs(features) do
-                               if not done[feature] then
-                                       reply:tag("feature", {var = feature}):up(); -- TODO cache
-                                       done[feature] = true;
-                               end
-                       end
-               end
-               for module, features in pairs(features_table:get("*") or NULL) do -- for each module
-                       for feature in pairs(features) do
-                               if not done[feature] then
-                                       reply:tag("feature", {var = feature}):up(); -- TODO cache
-                                       done[feature] = true;
-                               end
-                       end
-               end
-               return next(done) ~= nil;
-       end
-end);
-
 function api:add_feature(xmlns)
-       features_table:set(self.host, self.name, xmlns, true);
+       self:add_item("feature", xmlns);
+end
+function api:add_identity(category, type, name)
+       self:add_item("identity", {category = category, type = type, name = name});
 end
 
 local event_hook = function(host, mod_name, event_name, ...)
@@ -351,7 +351,77 @@ function api:add_event_hook(name, handler)
 end
 
 function api:fire_event(...)
-       return eventmanager.fire_event(...);
+       return (hosts[self.host] or prosody).events.fire_event(...);
+end
+
+function api:hook(event, handler, priority)
+       hooks:set(self.host, self.name, event, handler, true);
+       (hosts[self.host] or prosody).events.add_handler(event, handler, priority);
+end
+
+function api:hook_stanza(xmlns, name, handler, priority)
+       if not handler and type(name) == "function" then
+               -- If only 2 options then they specified no xmlns
+               xmlns, name, handler, priority = nil, xmlns, name, handler;
+       elseif not (handler and name) then
+               self:log("warn", "Error: Insufficient parameters to module:hook_stanza()");
+               return;
+       end
+       return api.hook(self, "stanza/"..(xmlns and (xmlns..":") or "")..name, function (data) return handler(data.origin, data.stanza, data); end, priority);
+end
+
+function api:require(lib)
+       local f, n = pluginloader.load_code(self.name, lib..".lib.lua");
+       if not f then
+               f, n = pluginloader.load_code(lib, lib..".lib.lua");
+       end
+       if not f then error("Failed to load plugin library '"..lib.."', error: "..n); end -- FIXME better error message
+       setfenv(f, setmetatable({ module = self }, { __index = _G }));
+       return f();
+end
+
+function api:get_option(name, default_value)
+       return config.get(self.host, self.name, name) or config.get(self.host, "core", name) or default_value;
+end
+
+local t_remove = _G.table.remove;
+local module_items = multitable_new();
+function api:add_item(key, value)
+       self.items = self.items or {};
+       self.items[key] = self.items[key] or {};
+       t_insert(self.items[key], value);
+       self:fire_event("item-added/"..key, {source = self, item = value});
+end
+function api:remove_item(key, value)
+       local t = self.items and self.items[key] or NULL;
+       for i = #t,1,-1 do
+               if t[i] == value then
+                       t_remove(self.items[key], i);
+                       self:fire_event("item-removed/"..key, {source = self, item = value});
+                       return value;
+               end
+       end
+end
+
+function api:get_host_items(key)
+       local result = {};
+       for mod_name, module in pairs(modulemap[self.host]) do
+               module = module.module;
+               if module.items then
+                       for _, item in ipairs(module.items[key] or NULL) do
+                               t_insert(result, item);
+                       end
+               end
+       end
+       for mod_name, module in pairs(modulemap["*"]) do
+               module = module.module;
+               if module.items then
+                       for _, item in ipairs(module.items[key] or NULL) do
+                               t_insert(result, item);
+                       end
+               end
+       end
+       return result;
 end
 
 --------------------------------------------------------------------