Merge lp:~stefanor/ibid/calc-525439 into lp:~ibid-core/ibid/old-trunk-1.6

[Stefano Rivera]

[Michael Gorven (mgorven)]

 review approve

review: Approve

[Michael Gorven (mgorven)]

Hackity hack...
 review approve

review: Approve

[lp:~stefanor/ibid/calc-525439]

894. By Stefano Rivera on 2010-02-23

Allow subclassing Processors

895. By Stefano Rivera on 2010-02-23

Subclass Clac cleanly

Unmerged revisions

=== modified file 'ibid/plugins/__init__.py'
--- ibid/plugins/__init__.py	2010-02-03 15:04:43 +0000
+++ ibid/plugins/__init__.py	2010-02-23 11:47:13 +0000
@@ -62,16 +62,17 @@
             new.default = getattr(cls, name)
             setattr(cls, name, new)
 
-        if getattr(cls, '_event_handlers', None) is None:
-            cls._event_handlers = []
-            for name, item in cls.__dict__.iteritems():
-                if getattr(item, 'handler', False):
-                    cls._event_handlers.append(name)
-        if getattr(cls, '_periodic_handlers', None) is None:
-            cls._periodic_handlers = []
-            for name, item in cls.__dict__.iteritems():
-                if getattr(item, 'periodic', False):
-                    cls._periodic_handlers.append(name)
+        for attr, listname in (
+                ('handler', 'event'),
+                ('periodic', 'periodic')):
+            listname = '_Processor__%s_handlers' % listname
+            if not hasattr(cls, listname):
+                setattr(cls, listname, [])
+            handlers = getattr(cls, listname)
+
+            for name, item in cls.__dict__.iteritems():
+                if getattr(item, attr, False) and name not in handlers:
+                    handlers.append(name)
 
         return super(Processor, cls).__new__(cls)
 
@@ -127,12 +128,12 @@
 
     def _get_event_handlers(self):
         "Find all the handlers (regex matching and blind)"
-        for handler in self._event_handlers:
+        for handler in self._event_handlers or self.__event_handlers:
             yield getattr(self, handler)
 
     def _get_periodic_handlers(self):
         "Find all the periodic handlers"
-        for handler in self._periodic_handlers:
+        for handler in self._periodic_handlers or self.__periodic_handlers:
             yield getattr(self, handler)
 
     def _run_periodic_handler(self, method, event):

=== modified file 'ibid/plugins/calc.py'
--- ibid/plugins/calc.py	2010-01-18 23:20:33 +0000
+++ ibid/plugins/calc.py	2010-02-23 11:47:13 +0000
@@ -2,10 +2,9 @@
 # Released under terms of the MIT/X/Expat Licence. See COPYING for details.
 
 from __future__ import division
-from random import random, randint
-
 import logging
 from os import kill
+from random import random, randint
 from signal import SIGTERM
 from subprocess import Popen, PIPE
 from time import time, sleep
@@ -151,7 +150,7 @@
     safe['pow'] = limited_pow
     safe['factorial'] = limited_factorial
 
-    @match(r'^(?:calc\s+)?(.+?)$')
+    @match(r'^(.+)$')
     def calculate(self, event, expression):
         for term in self.banned:
             if term in expression:
@@ -193,6 +192,14 @@
         if isinstance(result, (int, long, float, complex)):
             event.addresponse(unicode(result))
 
+
+class ExplicitCalc(Calc):
+    priority = 0
+
+    @match(r'^calc(?:ulate)?\s+(.+)$')
+    def calculate(self, event, expression):
+        super(ExplicitCalc, self).calculate(event, expression)
+
 help['random'] = u'Generates random numbers.'
 class Random(Processor):
     u"""random [ <max> | <min> <max> ]"""