From 2620b536feb081ffae43bd13b8ae401c510f1023 Mon Sep 17 00:00:00 2001
From: Giuseppe Bilotta <giuseppe.bilotta@gmail.com>
Date: Fri, 2 Feb 2007 16:50:58 +0000
Subject: [PATCH] Plugin map requirements are now checked at regular expression
 time, not later on

---
 ChangeLog                 |  6 +++++
 lib/rbot/messagemapper.rb | 55 ++++++++++++++++++---------------------
 2 files changed, 31 insertions(+), 30 deletions(-)

diff --git a/ChangeLog b/ChangeLog
index 0a6b03b3..3ca18500 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,9 @@
+2007-02-02  Giuseppe Bilotta <giuseppe.bilotta@gmail.com>
+
+	* Plugin message mapper: Requirements (as set by the :requirements
+	hash) are now embedded in the regular expression matcher rather than
+	being cheked separately afterwards.
+
 2007-01-30  Giuseppe Bilotta <giuseppe.bilotta@gmail.com>
 
 	* New IRC Framework: Server methods to retrieve a Channel or User are
diff --git a/lib/rbot/messagemapper.rb b/lib/rbot/messagemapper.rb
index f074dd20..c272ab77 100644
--- a/lib/rbot/messagemapper.rb
+++ b/lib/rbot/messagemapper.rb
@@ -277,8 +277,29 @@ module Irc
       rx = Regexp.escape(str)
       # debug "Escaped: #{rx.inspect}"
       rx.gsub!(/((?:\\ )*)(:|\\\*)(\w+)/) { |m|
-        not_needed = @defaults.has_key?($3.intern)
-        s = "#{not_needed ? "(?:" : ""}#{$1}(#{$2 == ":" ? "\\S+" : ".*"})#{ not_needed ? ")?" : ""}"
+        whites = $1
+        is_single = $2 == ":"
+        name = $3.intern
+
+        not_needed = @defaults.has_key?(name)
+
+        has_req = @requirements[name]
+        debug "Requirements for #{name}: #{has_req.inspect}"
+        case has_req
+        when nil
+          sub = is_single ? "\\S+" : ".*"
+        when Regexp
+          # Remove the ^ and $ placed around requirement regexp at times
+          # They were unnecessary first, and are dangerous now
+          sub = has_req.source.sub(/^\^/,'').sub(/\$$/,'')
+        when String
+          sub = Regexp.escape(has_req)
+        else
+          warning "Odd requirement #{has_req.inspect} of class #{has_req.class} for parameter '#{name}'"
+          sub = Regexp.escape(has_req.to_s) rescue "\\S+"
+        end
+        debug "Regexp for #{name}: #{sub.inspect}"
+        s = "#{not_needed ? "(?:" : ""}#{whites}(#{sub})#{ not_needed ? ")?" : ""}"
       }
       # debug "Replaced dyns: #{rx.inspect}"
       rx.gsub!(/((?:\\ )*)\\\[/, "(?:\\1")
@@ -343,19 +364,8 @@ module Irc
           options[item] = value
           debug "set #{item} to #{value.inspect}"
         else
-          if matching[i]
-            value = matching[i]
-            unless passes_requirements?(item, value)
-              # if @defaults.has_key?(item)
-              #   value = @defaults[item]
-              # else
-                return nil, requirements_for(item)
-              # end
-            end
-          else
-            value = @defaults[item]
-            warning "No default value for option #{item.inspect} specified" unless @defaults.has_key?(item)
-          end
+          value = matching[i] || value = @defaults[item]
+          warning "No default value for option #{item.inspect} specified" unless @defaults.has_key?(item)
           options[item] = value
           debug "set #{item} to #{options[item].inspect}"
         end
@@ -371,21 +381,6 @@ module Irc
       "<#{self.class.to_s} #{@items.map { |c| c.inspect }.join(' ').inspect}#{default_str}#{when_str}>"
     end
 
-    # Verify that the given value passes this template's requirements
-    def passes_requirements?(name, value)
-      return @defaults.key?(name) && @defaults[name].nil? if value.nil? # Make sure it's there if it should be
-
-      case @requirements[name]
-        when nil then true
-        when Regexp then
-          value = value.to_s
-          match = @requirements[name].match(value)
-          match && match[0].length == value.length
-        else
-          @requirements[name] == value.to_s
-      end
-    end
-
     def requirements_for(name)
       name = name.to_s.sub(/^\*/,"").intern if (/^\*/ =~ name.inspect)
       presence = (@defaults.key?(name) && @defaults[name].nil?)
-- 
2.39.2