Fixed Bug #833786 (spurious brackets not handled correctly).

src/Owner.py

@@ -85,7 +85,10 @@ class Owner(privmsgs.CapabilityCheckingPrivmsg):
     def __init__(self):
         callbacks.Privmsg.__init__(self)
         setattr(self.__class__, 'exec', self.__class__._exec)
-        self.defaultPlugins = {'join', 'admin'}
+        self.defaultPlugins = {'join': 'admin',
+                               'load': 'owner',
+                               'reload': 'owner',
+                               'unload': 'owner'}
 
     def _disambiguate(self, tokens):
         if tokens:
@@ -100,7 +103,11 @@ class Owner(privmsgs.CapabilityCheckingPrivmsg):
         callbacks.Privmsg.handled = False
         s = callbacks.addressed(irc.nick, msg)
         if s:
-            tokens = callbacks.tokenize(s)
+            try:
+                tokens = callbacks.tokenize(s)
+            except SyntaxError, e:
+                irc.queueMsg(callbacks.error(msg, str(e)))
+                return
             self._disambiguate(tokens)
             ambiguousCommands = {}
             commands = callbacks.getCommands(tokens)

test/test_callbacks.py

@@ -296,7 +296,10 @@ class PrivmsgTestCase(ChannelPluginTestCase):
             self.assertEqual(conf.prefixChars, '$')
         finally:
             conf.commandsOnStart = original
-            
+
+    def testSyntaxErrorNotEscaping(self):
+        self.assertError('load [foo')
+        self.assertError('load foo]')
 
 
 class PrivmsgCommandAndRegexpTestCase(PluginTestCase):