From 8c76f24c654c813fe9510a2d1b06d635faced6fe Mon Sep 17 00:00:00 2001 From: jesopo Date: Thu, 13 Feb 2020 10:13:39 +0000 Subject: [PATCH] actually raise exception for !modinfo not found --- src/core_modules/modules.py | 10 ++++++---- 1 file changed, 6 insertions(+), 4 deletions(-) diff --git a/src/core_modules/modules.py b/src/core_modules/modules.py index f6d19fe3..d4c40729 100644 --- a/src/core_modules/modules.py +++ b/src/core_modules/modules.py @@ -3,6 +3,8 @@ from src import ModuleManager, utils +ERR_NOTLOADED = "Module '%s' isn't loaded" + class Module(ModuleManager.BaseModule): def _catch(self, name, func): try: @@ -10,7 +12,7 @@ class Module(ModuleManager.BaseModule): except ModuleManager.ModuleNotFoundException: raise utils.EventError("Module '%s' not found" % name) except ModuleManager.ModuleNotLoadedException: - raise utils.EventError("Module '%s' isn't loaded" % name) + raise utils.EventError(ERR_NOTLOADED % name) except ModuleManager.ModuleWarning as warning: raise utils.EventError("Module '%s' not loaded: %s" % ( name, str(warning))) @@ -19,11 +21,11 @@ class Module(ModuleManager.BaseModule): name, str(e))) @utils.hook("received.command.modinfo") - @utils.spec("!string") + @utils.spec("!word") def info(self, event): name = event["spec"][0] if not name in self.bot.modules.modules: - raise + raise utils.EventError(ERR_NOTLOADED % name) module = self.bot.modules.modules[name] event_calls = 0 @@ -63,7 +65,7 @@ class Module(ModuleManager.BaseModule): def unload(self, event): name = event["spec"][0] if not name in self.bot.modules.modules: - raise utils.EventError("Module '%s' isn't loaded" % name) + raise utils.EventError(ERR_NOTLOADED % name) self._catch(name, lambda: self.bot.modules.unload_module(name)) event["stdout"].write("Unloaded '%s'" % name)