aboutsummaryrefslogtreecommitdiffstats
path: root/mediagoblin
diff options
context:
space:
mode:
authorSebastian Spaeth <Sebastian@SSpaeth.de>2012-12-12 13:53:56 +0100
committerSebastian Spaeth <Sebastian@SSpaeth.de>2013-01-16 20:14:55 +0100
commitf91dcc9d963c87d6b6752cb397e1ff9fdf57ad28 (patch)
tree50648548fc21c02f509c37f2c92246b0c2d21951 /mediagoblin
parentc11c1994e61d79c0332f6782a589ea012ecf2473 (diff)
downloadmediagoblin-f91dcc9d963c87d6b6752cb397e1ff9fdf57ad28.tar.lz
mediagoblin-f91dcc9d963c87d6b6752cb397e1ff9fdf57ad28.tar.xz
mediagoblin-f91dcc9d963c87d6b6752cb397e1ff9fdf57ad28.zip
Implement @get_workbench decorator
This passes in a Workbench() via the 'workbench' keyword argument, and conveniently cleans it up after the function has finished. 2 out of our 5 backends forgot to clean up their workbench, so this is clearly needed :-). Signed-off-by: Sebastian Spaeth <Sebastian@SSpaeth.de>
Diffstat (limited to 'mediagoblin')
-rw-r--r--mediagoblin/decorators.py12
1 files changed, 12 insertions, 0 deletions
diff --git a/mediagoblin/decorators.py b/mediagoblin/decorators.py
index a40f1d5a..3655d758 100644
--- a/mediagoblin/decorators.py
+++ b/mediagoblin/decorators.py
@@ -20,6 +20,7 @@ from urlparse import urljoin
from werkzeug.exceptions import Forbidden, NotFound
from werkzeug.urls import url_quote
+from mediagoblin import mg_globals as mgg
from mediagoblin.db.models import MediaEntry, User
from mediagoblin.tools.response import redirect, render_404
@@ -222,3 +223,14 @@ def get_media_entry_by_id(controller):
return controller(request, media=media, *args, **kwargs)
return wrapper
+
+
+def get_workbench(func):
+ """Decorator, passing in a workbench as kwarg which is cleaned up afterwards"""
+
+ @wraps(func)
+ def new_func(*args, **kwargs):
+ with mgg.workbench_manager.create_workbench() as workbench:
+ return func(*args, workbench=workbench, **kwargs)
+
+ return new_func