]> git.netwichtig.de Git - user/henk/code/ruby/rbot.git/commitdiff
message.rb: fix the fix to message addressing logic
authorGiuseppe Bilotta <giuseppe.bilotta@gmail.com>
Fri, 5 Sep 2008 17:35:49 +0000 (19:35 +0200)
committerGiuseppe Bilotta <giuseppe.bilotta@gmail.com>
Fri, 5 Sep 2008 17:39:33 +0000 (19:39 +0200)
This reverts commit b6bd9a24014ac9fbc822e9051f216f0506f24cd5
"message.rb: fix message addressing logic" since it actually breaks the
validity for address? in a number of cases.

Instead, add proper @address setting to NickMessage which is the only
one getting it wrong presently.

lib/rbot/message.rb

index f6e38e80a73d8f2359f38508466807b51b30c7b4..b34105c45aff3bb699903c0e8533b654b32f421b 100644 (file)
@@ -195,7 +195,9 @@ module Irc
       @plainmessage = BasicUserMessage.strip_formatting(@message)
       @message = BasicUserMessage.strip_initial_formatting(@message)
 
-      @address = true if source == @bot.myself
+      if target && target == @bot.myself
+        @address = true
+      end
 
     end
 
@@ -321,8 +323,9 @@ module Irc
       @ctcp = false
       @action = false
 
-      if @address = (target == @bot.myself)
+      if target == @bot.myself
         @private = true
+        @address = true
         @channel = nil
         @replyto = source
       else
@@ -517,6 +520,7 @@ module Irc
     attr_accessor :is_on
     def initialize(bot, server, source, oldnick, newnick)
       super(bot, server, source, oldnick, newnick)
+      @address = (source == @bot.myself)
       @is_on = []
     end
 
@@ -540,6 +544,7 @@ module Irc
     attr_accessor :modes
     def initialize(bot, server, source, target, message="")
       super(bot, server, source, target, message)
+      @address = (source == @bot.myself)
       @modes = []
     end
 
@@ -554,6 +559,7 @@ module Irc
     attr_reader :whois
     def initialize(bot, server, source, target, whois)
       super(bot, server, source, target, "")
+      @address = (target == @bot.myself)
       @whois = whois
     end
 
@@ -624,6 +630,7 @@ module Irc
       super(bot, server, source, channel, message)
       @channel = channel
       # in this case sourcenick is the nick that could be the bot
+      @address = (source == @bot.myself)
     end
   end