mod_storage_sql: Add method for listing stores
[prosody.git] / core / modulemanager.lua
index bf1e1924acbb59b49010604b1cb4cc1991a0cb09..9c5e4a4a276c8b06c9d0a11ca91a89e35394c3a7 100644 (file)
@@ -10,19 +10,16 @@ local logger = require "util.logger";
 local log = logger.init("modulemanager");
 local config = require "core.configmanager";
 local pluginloader = require "util.pluginloader";
+local set = require "util.set";
+
+local new_multitable = require "util.multitable".new;
 
 local hosts = hosts;
 local prosody = prosody;
 
-local loadfile, pcall, xpcall = loadfile, pcall, xpcall;
-local setmetatable, setfenv, getfenv = setmetatable, setfenv, getfenv;
-local pairs, ipairs = pairs, ipairs;
-local t_insert, t_concat = table.insert, table.concat;
-local type = type;
-local next = next;
-local rawget = rawget;
-local error = error;
-local tostring, tonumber = tostring, tonumber;
+local pcall, xpcall = pcall, xpcall;
+local setmetatable, rawget = setmetatable, rawget;
+local pairs, type, tostring = pairs, type, tostring;
 
 local debug_traceback = debug.traceback;
 local unpack, select = unpack, select;
@@ -32,10 +29,8 @@ pcall = function(f, ...)
        return xpcall(function() return f(unpack(params, 1, n)) end, function(e) return tostring(e).."\n"..debug_traceback(); end);
 end
 
-local array, set = require "util.array", require "util.set";
-
 local autoload_modules = {"presence", "message", "iq", "offline", "c2s", "s2s"};
-local component_inheritable_modules = {"tls", "dialback", "iq"};
+local component_inheritable_modules = {"tls", "dialback", "iq", "s2s"};
 
 -- We need this to let modules access the real global namespace
 local _G = _G;
@@ -47,8 +42,6 @@ local api = _G.require "core.moduleapi"; -- Module API container
 -- [host] = { [module] = module_env }
 local modulemap = { ["*"] = {} };
 
-local NULL = {};
-
 -- Load modules when a host is activated
 function load_modules_for_host(host)
        local component = config.get(host, "core", "component_module");
@@ -82,6 +75,9 @@ function load_modules_for_host(host)
        end
 end
 prosody.events.add_handler("host-activated", load_modules_for_host);
+prosody.events.add_handler("host-deactivated", function (host)
+       modulemap[host] = nil;
+end);
 
 --- Private helpers ---
 
@@ -96,8 +92,8 @@ local function do_unload_module(host, name)
                end
        end
        
-       for handler, event in pairs(mod.module.event_handlers) do
-               event.object.remove_handler(event.name, handler);
+       for object, event, handler in mod.module.event_handlers:iter(nil, nil, nil) do
+               object.remove_handler(event, handler);
        end
        
        if mod.module.items then -- remove items
@@ -137,38 +133,43 @@ local function do_load_module(host, module_name)
                if module_has_method(mod, "add_host") then
                        local _log = logger.init(host..":"..module_name);
                        local host_module_api = setmetatable({
-                               host = host, event_handlers = {}, items = {};
+                               host = host, event_handlers = new_multitable(), items = {};
                                _log = _log, log = function (self, ...) return _log(...); end;
                        },{
                                __index = modulemap["*"][module_name].module;
                        });
-                       local ok, result, module_err = call_module_method(mod, "add_host", host_module_api);
-                       if not ok or result == false then return nil, ok and module_err or result; end
                        local host_module = setmetatable({ module = host_module_api }, { __index = mod });
-                       host_module.module.environment = host_module;
+                       host_module_api.environment = host_module;
                        modulemap[host][module_name] = host_module;
+                       local ok, result, module_err = call_module_method(mod, "add_host", host_module_api);
+                       if not ok or result == false then
+                               modulemap[host][module_name] = nil;
+                               return nil, ok and module_err or result;
+                       end
                        return host_module;
                end
                return nil, "global-module-already-loaded";
        end
        
 
-       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;
-       end
 
        local _log = logger.init(host..":"..module_name);
-       local api_instance = setmetatable({ name = module_name, host = host, path = err,
-               _log = _log, log = function (self, ...) return _log(...); end, event_handlers = {} }
+       local api_instance = setmetatable({ name = module_name, host = host,
+               _log = _log, log = function (self, ...) return _log(...); end, event_handlers = new_multitable() }
                , { __index = api });
 
        local pluginenv = setmetatable({ module = api_instance }, { __index = _G });
        api_instance.environment = pluginenv;
        
-       setfenv(mod, pluginenv);
-       
+       local mod, err = pluginloader.load_code(module_name, nil, pluginenv);
+       if not mod then
+               log("error", "Unable to load module '%s': %s", module_name or "nil", err or "nil");
+               return nil, err;
+       end
+
+       api_instance.path = err;
+
+       modulemap[host][module_name] = pluginenv;
        local ok, err = pcall(mod);
        if ok then
                -- Call module's "load"
@@ -179,12 +180,15 @@ local function do_load_module(host, module_name)
                        end
                end
 
-               modulemap[api_instance.host][module_name] = pluginenv;
                if api_instance.host == "*" then
                        if not api_instance.global then -- COMPAT w/pre-0.9
-                               log("warn", "mod_%s: Setting module.host = '*' deprecated, call module:set_global() instead", module_name);
+                               if host ~= "*" then
+                                       log("warn", "mod_%s: Setting module.host = '*' deprecated, call module:set_global() instead", module_name);
+                               end
                                api_instance:set_global();
                        end
+                       modulemap[host][module_name] = nil;
+                       modulemap[api_instance.host][module_name] = pluginenv;
                        if host ~= api_instance.host and module_has_method(pluginenv, "add_host") then
                                -- Now load the module again onto the host it was originally being loaded on
                                ok, err = do_load_module(host, module_name);
@@ -192,6 +196,7 @@ local function do_load_module(host, module_name)
                end
        end
        if not ok then
+               modulemap[api_instance.host][module_name] = nil;
                log("error", "Error initializing module '%s' on '%s': %s", module_name, host, err or "nil");
        end
        return ok and pluginenv, err;
@@ -243,7 +248,7 @@ end
 function load(host, name)
        local mod, err = do_load_module(host, name);
        if mod then
-               (hosts[mod.module.host] or prosody).events.fire_event("module-loaded", { module = name, host = host });
+               (hosts[mod.module.host] or prosody).events.fire_event("module-loaded", { module = name, host = mod.module.host });
        end
        return mod, err;
 end
@@ -258,13 +263,15 @@ function unload(host, name)
 end
 
 function reload(host, name)
-       local ok, err = do_reload_module(host, name);
-       if ok then
+       local mod, err = do_reload_module(host, name);
+       if mod then
+               modulemap[host][name].module.reloading = true;
                (hosts[host] or prosody).events.fire_event("module-reloaded", { module = name, host = host });
+               mod.module.reloading = nil;
        elseif not is_loaded(host, name) then
                (hosts[host] or prosody).events.fire_event("module-unloaded", { module = name, host = host });
        end
-       return ok, err;
+       return mod, err;
 end
 
 function get_module(host, name)