From 33c861b4e1fb88c52585647a958ac27810399704 Mon Sep 17 00:00:00 2001 From: Numerlor <25886452+Numerlor@users.noreply.github.com> Date: Sat, 9 Jan 2021 19:27:21 +0100 Subject: Do not add package name to the front of the symbol if it's already there --- bot/exts/info/doc/_cog.py | 13 +++++++++++-- 1 file changed, 11 insertions(+), 2 deletions(-) diff --git a/bot/exts/info/doc/_cog.py b/bot/exts/info/doc/_cog.py index 7aa6d0428..feb08e1cb 100644 --- a/bot/exts/info/doc/_cog.py +++ b/bot/exts/info/doc/_cog.py @@ -319,9 +319,18 @@ class DocCog(commands.Cog): self.doc_symbols[overridden_symbol] = original_item self.renamed_symbols.add(overridden_symbol) + # If we can't specially handle the symbol through its group or package, + # fall back to prepending its package name to the front. else: - self.renamed_symbols.add(symbol_name) - return f"{package_name}.{symbol_name}" + if symbol_name.startswith(package_name): + # If the symbol already starts with the package name, insert the group name after it. + split_symbol_name = symbol_name.split(".", maxsplit=1) + split_symbol_name.insert(1, group_name) + overridden_symbol = ".".join(split_symbol_name) + else: + overridden_symbol = f"{package_name}.{symbol_name}" + self.renamed_symbols.add(overridden_symbol) + return overridden_symbol async def refresh_inventory(self) -> None: """Refresh internal documentation inventory.""" -- cgit v1.2.3