From 84e59b05ff0a17178da9ecdb6c5d084e48b42763 Mon Sep 17 00:00:00 2001 From: mathieui Date: Sun, 21 Aug 2016 15:27:53 +0200 Subject: =?UTF-8?q?Don=E2=80=99t=20call=20input=20completion()=20functions?= =?UTF-8?q?=20inside=20completion=20methods?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Use a placeholder object that can run it afterwards, so that we don’t have side effects inside those functions. --- plugins/admin.py | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) (limited to 'plugins/admin.py') diff --git a/plugins/admin.py b/plugins/admin.py index 01672014..8c632532 100644 --- a/plugins/admin.py +++ b/plugins/admin.py @@ -53,6 +53,7 @@ For affiliations from poezio.plugin import BasePlugin from poezio.tabs import MucTab +from poezio.core.structs import Completion class Plugin(BasePlugin): """ @@ -113,7 +114,7 @@ class Plugin(BasePlugin): compare_users = lambda x: x.last_talked word_list = [user.nick for user in sorted(tab.users, key=compare_users, reverse=True)\ if user.nick != tab.own_nick] - return the_input.auto_completion(word_list, '') + return Completion(the_input.auto_completion, word_list, '') -- cgit v1.2.3