diff options
author | Sebastian Spaeth <Sebastian@SSpaeth.de> | 2012-12-12 13:53:56 +0100 |
---|---|---|
committer | Sebastian Spaeth <Sebastian@SSpaeth.de> | 2013-01-16 20:14:55 +0100 |
commit | f91dcc9d963c87d6b6752cb397e1ff9fdf57ad28 (patch) | |
tree | 50648548fc21c02f509c37f2c92246b0c2d21951 /mediagoblin | |
parent | c11c1994e61d79c0332f6782a589ea012ecf2473 (diff) | |
download | mediagoblin-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.py | 12 |
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 |