Don’t call input completion() functions inside completion methods

Use a placeholder object that can run it afterwards, so that we don’t
have side effects inside those functions.

1 files changed, 3 insertions, 2 deletions

diff --git a/plugins/irc.py b/plugins/irc.py
index 56ce5c9c..263d7d22 100644
--- a/plugins/irc.py
+++ b/plugins/irc.py
@@ -131,6 +131,7 @@ Example configuration
 
 from poezio.plugin import BasePlugin
 from poezio.decorators import command_args_parser
+from poezio.core.structs import Completion
 from poezio import common
 from poezio import tabs
 
@@ -270,7 +271,7 @@ class Plugin(BasePlugin):
                 sections.remove(section)
             except:
                 pass
-        return the_input.new_completion(sections, pos)
+        return Completion(the_input.new_completion, sections, pos)
 
     @command_args_parser.quoted(1, 1)
     def command_irc_join(self, args):
@@ -375,6 +376,6 @@ class Plugin(BasePlugin):
         sections = self.config.sections()
         if 'irc' in sections:
             sections.remove('irc')
-        return the_input.new_completion(sections, 1)
+        return Completion(the_input.new_completion, sections, 1)