Tag addressed messages with the results of callbacks._addressed. Also,

reorder the calling of addressedRes and res in PrivmsgCommandAndRegexp so
that addressedRes are run first.

src/callbacks.py

@@ -64,7 +64,7 @@ import supybot.ircmsgs as ircmsgs
 import supybot.ircutils as ircutils
 import supybot.registry as registry
 
-def addressed(nick, msg, prefixChars=None, nicks=None,
+def _addressed(nick, msg, prefixChars=None, nicks=None,
               prefixStrings=None, whenAddressedByNick=None,
               whenAddressedByNickAtEnd=None):
     """If msg is addressed to 'name', returns the portion after the address.
@@ -144,6 +144,15 @@ def addressed(nick, msg, prefixChars=None, nicks=None,
     else:
         return ''
 
+def addressed(nick, msg, **kwargs):
+    payload = msg.addressed
+    if payload is not None:
+        return payload
+    else:
+        payload = _addressed(nick, msg, **kwargs)
+        msg.tag('addressed', payload)
+        return payload
+
 def canonicalName(command):
     """Turn a command into its canonical form.
 
@@ -1364,10 +1373,6 @@ class PrivmsgCommandAndRegexp(Privmsg):
         if msg.isError:
             self.log.debug('%s not running due to msg.isError.', self.name())
             return
-        for (r, name) in self.res:
-            for m in r.finditer(msg.args[1]):
-                proxy = self.Proxy(irc, msg)
-                self.callCommand(name, proxy, msg, m, catchErrors=True)
         s = addressed(irc.nick, msg)
         if s:
             for (r, name) in self.addressedRes:
@@ -1376,6 +1381,10 @@ class PrivmsgCommandAndRegexp(Privmsg):
                 for m in r.finditer(s):
                     proxy = self.Proxy(irc, msg)
                     self.callCommand(name, proxy, msg, m, catchErrors=True)
+        for (r, name) in self.res:
+            for m in r.finditer(msg.args[1]):
+                proxy = self.Proxy(irc, msg)
+                self.callCommand(name, proxy, msg, m, catchErrors=True)
 
 
 # vim:set shiftwidth=4 tabstop=8 expandtab textwidth=78:

test/test_callbacks.py

@@ -196,6 +196,10 @@ class FunctionsTestCase(SupyTestCase):
         try:
             original = conf.supybot.reply.whenNotAddressed()
             conf.supybot.reply.whenNotAddressed.setValue(True)
+            # need to recreate the msg objects since the old ones have already
+            # been tagged
+            msg1 = ircmsgs.privmsg('#foo', '@bar')
+            msg2 = ircmsgs.privmsg('#foo', 'bar')
             self.assertEqual(callbacks.addressed('blah', msg1), 'bar')
             self.assertEqual(callbacks.addressed('blah', msg2), 'bar')
         finally:
@@ -204,6 +208,9 @@ class FunctionsTestCase(SupyTestCase):
     def testAddressedWithMultipleNicks(self):
         msg = ircmsgs.privmsg('#foo', 'bar: baz')
         self.assertEqual(callbacks.addressed('bar', msg), 'baz')
+        # need to recreate the msg objects since the old ones have already
+        # been tagged
+        msg = ircmsgs.privmsg('#foo', 'bar: baz')
         self.assertEqual(callbacks.addressed('biff', msg, nicks=['bar']),
                          'baz')