X-Git-Url: https://git.openstreetmap.org./osqa.git/blobdiff_plain/410bfa05ee36ed1d99356c443a5f3f6aa3ee9578..162b4fd442c169a61d2f5b2b3bd613546c8472da:/forum/modules/decorators.py

diff --git a/forum/modules/decorators.py b/forum/modules/decorators.py
index 17d248b..c7c5640 100644
--- a/forum/modules/decorators.py
+++ b/forum/modules/decorators.py
@@ -1,53 +1,128 @@
 import inspect
 
 class DecoratableObject(object):
+    MODE_OVERRIDE = 0
+    MODE_PARAMS = 1
+    MODE_RESULT = 2
+
     def __init__(self, fn):
+        a = inspect.getargspec(fn)
         self._callable = fn
+        self._params_decoration = None
+        self._result_decoration = None
 
-    def decorate(self, fn, needs_origin):
+    def _decorate(self, fn, needs_origin, method=False):
         origin = self._callable
 
         if needs_origin:
-            self._callable = lambda *args, **kwargs: fn(origin, *args, **kwargs)
+            if method:
+                self._callable = lambda inst, *args, **kwargs: fn(inst, origin, *args, **kwargs)
+            else:
+                self._callable = lambda *args, **kwargs: fn(origin, *args, **kwargs)
         else:
-            self._callable = lambda *args, **kwargs: fn(*args, **kwargs)
+            self._callable = fn
+
+    def _decorate_params(self, fn):
+        if not self._params_decoration:
+            self._params_decoration = []
+
+        self._params_decoration.append(fn)
+
+    def _decorate_result(self, fn):
+        if not self._result_decoration:
+            self._result_decoration = []
+
+        self._result_decoration.append(fn)
 
     def __call__(self, *args, **kwargs):
-        return self._callable(*args, **kwargs)
+        if self._params_decoration:
+            for dec in self._params_decoration:
+                args, kwargs = dec(*args, **kwargs)
+
+        res = self._callable(*args, **kwargs)
 
+        if self._result_decoration:
+            for dec in self._result_decoration:
+                res = dec(res)
 
-def decoratable(fn):
-    return DecoratableObject(fn)
+        return res
+
+
+def _create_decorator(origin, needs_origin, mode, method=False):
+    def decorator(fn):
+        if mode == DecoratableObject.MODE_OVERRIDE:
+            origin._decorate(fn, needs_origin, method=method)
+        elif mode == DecoratableObject.MODE_PARAMS:
+            origin._decorate_params(fn)
+        elif mode == DecoratableObject.MODE_RESULT:
+            origin._decorate_result(fn)
+
+        return fn
+
+    return decorator
 
-def decoratable_method(fn):
-    obj = DecoratableObject(fn)
-    def decorated(self, *args, **kwargs):
-        return obj(self, *args, **kwargs)
 
-    decorated.__obj = obj
-    return decorated
+def _decorate_method(origin, needs_origin, mode):
+    if not hasattr(origin, '_decoratable_obj'):
+        name = origin.__name__
+        cls = origin.im_class
 
-decoratable.method = decoratable_method
+        decoratable = DecoratableObject(origin)
 
-def decorate(origin, needs_origin=True):
+        def decoratable_method(self, *args, **kwargs):
+            return decoratable(self, *args, **kwargs)
+
+        decoratable_method._decoratable_obj = decoratable
+        setattr(cls, name, decoratable_method)
+    else:
+        decoratable = origin._decoratable_obj
+
+    return _create_decorator(decoratable, needs_origin, mode, method=True)
+
+def _decorate_function(origin, needs_origin, mode):
     if not isinstance(origin, DecoratableObject):
-        if hasattr(origin, '__obj'):
-            def decorator(fn):
-                origin.__obj.decorate(fn, needs_origin)
-                return origin
-            return decorator
+        mod = inspect.getmodule(origin)
+
+        name = origin.__name__
+        origin = DecoratableObject(origin)
+        setattr(mod, name, origin)
+
+    return _create_decorator(origin, needs_origin, mode)
+
 
-        raise Exception('Not an decoratable function: %s' % origin.name)
+def decorate(origin, needs_origin=True, mode=DecoratableObject.MODE_OVERRIDE):
+    if inspect.ismethod(origin):
+        return _decorate_method(origin, needs_origin, mode)
+
+    if inspect.isfunction(origin) or isinstance(origin, DecoratableObject):
+        return _decorate_function(origin, needs_origin, mode)
 
     def decorator(fn):
-        origin.decorate(fn, needs_origin)
-        return origin
+        return fn
 
     return decorator
 
 
+def _decorate_params(origin):
+    return decorate(origin, mode=DecoratableObject.MODE_PARAMS)
+
+decorate.params = _decorate_params
 
+def _decorate_result(origin):
+    return decorate(origin, mode=DecoratableObject.MODE_RESULT)
 
+decorate.result = _decorate_result
 
+def _decorate_with(fn):
+    def decorator(origin):
+        if not isinstance(origin, DecoratableObject):
+            decoratable = DecoratableObject(origin)
+        else:
+            decoratable = origin
+
+        decoratable._decorate(fn, True, False)
+        return decoratable
+    return decorator
 
 
+decorate.withfn = _decorate_with
\ No newline at end of file