From: Giuseppe Bilotta <giuseppe.bilotta@gmail.com>
Date: Sun, 29 Jun 2008 18:19:41 +0000 (+0200)
Subject: * fix handling of IDENTIFY_MSG
X-Git-Url: https://git.netwichtig.de/gitweb/?a=commitdiff_plain;h=123fc3dd723a044e729341eaedfb8cc75390ac84;p=user%2Fhenk%2Fcode%2Fruby%2Frbot.git

* fix handling of IDENTIFY_MSG

When a server has IDENTIFY-MSG, we would expect identification in any
PRIVMSG or NOTICE, even on those generated from the bot. This caused
lots of spurious warnings, and would lead to mislogging when a
bot-generated message started with + or -.

Fix this by only handling IDENTIFY-MSG on server-generated messages.
---

diff --git a/lib/rbot/ircbot.rb b/lib/rbot/ircbot.rb
index 403f496d..91c383a5 100644
--- a/lib/rbot/ircbot.rb
+++ b/lib/rbot/ircbot.rb
@@ -615,7 +615,7 @@ class Bot
     }
 
     @client[:privmsg] = proc { |data|
-      m = PrivMessage.new(self, server, data[:source], data[:target], data[:message])
+      m = PrivMessage.new(self, server, data[:source], data[:target], data[:message], :handle_id => true)
       # debug "Message source is #{data[:source].inspect}"
       # debug "Message target is #{data[:target].inspect}"
       # debug "Bot is #{myself.inspect}"
@@ -629,7 +629,7 @@ class Bot
       @plugins.irc_delegate('privmsg', m)
     }
     @client[:notice] = proc { |data|
-      message = NoticeMessage.new(self, server, data[:source], data[:target], data[:message])
+      message = NoticeMessage.new(self, server, data[:source], data[:target], data[:message], :handle_id => true)
       # pass it off to plugins that want to hear everything
       @plugins.irc_delegate "notice", message
     }
diff --git a/lib/rbot/message.rb b/lib/rbot/message.rb
index 901626ec..6354ea11 100644
--- a/lib/rbot/message.rb
+++ b/lib/rbot/message.rb
@@ -467,17 +467,17 @@ module Irc
 
   # class to manage IRC PRIVMSGs
   class PrivMessage < UserMessage
-    def initialize(bot, server, source, target, message)
-      @msg_wants_id = true
-      super
+    def initialize(bot, server, source, target, message, opts={})
+      @msg_wants_id = opts[:handle_id]
+      super(bot, server, source, target, message)
     end
   end
 
   # class to manage IRC NOTICEs
   class NoticeMessage < UserMessage
-    def initialize(bot, server, source, target, message)
-      @msg_wants_id = true
-      super
+    def initialize(bot, server, source, target, message, opts={})
+      @msg_wants_id = opts[:handle_id]
+      super(bot, server, source, target, message)
     end
   end