From 98dde0dfe318d01e108acaed2413cc842a907b12 Mon Sep 17 00:00:00 2001 From: Yaohan Chen Date: Mon, 23 Jul 2007 19:14:37 +0000 Subject: [PATCH] * Prevent the alias from including arguments not substituded in command, as a check for possible incorrect command usage --- data/rbot/plugins/alias.rb | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/data/rbot/plugins/alias.rb b/data/rbot/plugins/alias.rb index 900df2c4..2fba9324 100644 --- a/data/rbot/plugins/alias.rb +++ b/data/rbot/plugins/alias.rb @@ -29,6 +29,7 @@ # to detect and stop this, but a few recursive calls can still cause spamming require 'yaml' +require 'set' class AliasPlugin < Plugin # an exception raised when loading or getting input of invalid alias definitions @@ -103,8 +104,9 @@ class AliasPlugin < Plugin # each alias is implemented by adding a message map, whose handler creates a message # containing the aliased command - command.grep(/<(\w+)>/) {$1}.all? {|s| text =~ /(?:^|\s)[:*]#{s}(?:\s|$)/ } or - raise AliasDefinitionError.new(_('Not all substitutions in command text have matching arguments in alias text')) + command.grep(/<(\w+)>/) {$1}.to_set == + text.grep(/(?:^|\s)[:*](\w+)(?:\s|$)/) {$1}.to_set or + raise AliasDefinitionError.new(_('The arguments in alias must match the substitutions in command, and vice versa')) @aliases[text] = command map text, :action => :"alias_handle<#{text}>", :auth_path => 'run' -- 2.39.5