Clearer logging for add_module_alias
Also, don't report anything if attempting to create an alias that already exists
This commit is contained in:
parent
fb20033900
commit
911c18eda7
|
@ -117,14 +117,16 @@ class BotModule(ABC):
|
||||||
"""
|
"""
|
||||||
for name in args:
|
for name in args:
|
||||||
if bot.modules.get(name):
|
if bot.modules.get(name):
|
||||||
self.logger.info(f"aliases: {name} is already a module!")
|
self.logger.info(f"not aliasing {name}, it is already a module")
|
||||||
continue
|
continue
|
||||||
prev = bot.module_aliases.get(name)
|
prev = bot.module_aliases.get(name)
|
||||||
|
if prev == self.name:
|
||||||
|
continue
|
||||||
if prev and not force:
|
if prev and not force:
|
||||||
self.logger.info(f"aliases: {name} already exists for module {prev}")
|
self.logger.info(f"not aliasing {name}, it is already an alias for {prev}")
|
||||||
continue
|
continue
|
||||||
if prev:
|
if prev:
|
||||||
self.logger.info(f"aliases: {name} from module {prev}")
|
self.logger.debug(f"overriding alias {name} for {prev}")
|
||||||
bot.module_aliases[name] = self.name
|
bot.module_aliases[name] = self.name
|
||||||
|
|
||||||
def enable(self):
|
def enable(self):
|
||||||
|
|
Loading…
Reference in New Issue