Fix aliases not being shown for disambiguation in utils.at_search_result()
Also correct misplaced 'if' used to avoid '[]' in the case of no aliases and update unit tests.
This commit is contained in:
parent
a1023ebc7e
commit
2ec97f102d
2 changed files with 16 additions and 6 deletions
|
|
@ -2403,9 +2403,9 @@ def at_search_result(matches, caller, query="", quiet=False, **kwargs):
|
|||
aliases = result.aliases.all(return_objs=True)
|
||||
# remove pluralization aliases
|
||||
aliases = [
|
||||
alias
|
||||
alias.db_key
|
||||
for alias in aliases
|
||||
if hasattr(alias, "category") and alias.category not in ("plural_key",)
|
||||
if alias.db_category != "plural_key"
|
||||
]
|
||||
else:
|
||||
# result is likely a Command, where `.aliases` is a list of strings.
|
||||
|
|
@ -2416,7 +2416,7 @@ def at_search_result(matches, caller, query="", quiet=False, **kwargs):
|
|||
name=result.get_display_name(caller)
|
||||
if hasattr(result, "get_display_name")
|
||||
else query,
|
||||
aliases=" [{alias}]".format(alias=";".join(aliases) if aliases else ""),
|
||||
aliases=" [{alias}]".format(alias=";".join(aliases)) if aliases else "",
|
||||
info=result.get_extra_info(caller),
|
||||
)
|
||||
matches = None
|
||||
|
|
|
|||
Loading…
Add table
Add a link
Reference in a new issue