aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorSebastian Spaeth <Sebastian@SSpaeth.de>2013-12-12 17:38:18 +0100
committerSebastian Spaeth <Sebastian@SSpaeth.de>2013-12-12 17:38:18 +0100
commit686cbcd974b464839b09338bab2bc11040c1043b (patch)
tree135a0b3207d0bf86a58d478dda8d054b85f62940
parent52a0387fc6ad023516c765019725ca58daaae347 (diff)
downloadmediagoblin-686cbcd974b464839b09338bab2bc11040c1043b.tar.lz
mediagoblin-686cbcd974b464839b09338bab2bc11040c1043b.tar.xz
mediagoblin-686cbcd974b464839b09338bab2bc11040c1043b.zip
Undo changes since gsoc in media_confirm_delete
Since the Gsoc the use of decorators has simplified this function. Reintroduce the same code as before Adidits blog branch made these changes.
-rw-r--r--mediagoblin/user_pages/views.py26
1 files changed, 4 insertions, 22 deletions
diff --git a/mediagoblin/user_pages/views.py b/mediagoblin/user_pages/views.py
index 8adcd554..64fa793e 100644
--- a/mediagoblin/user_pages/views.py
+++ b/mediagoblin/user_pages/views.py
@@ -286,29 +286,11 @@ def media_collect(request, media):
#TODO: Why does @user_may_delete_media not implicate @require_active_login?
-
+@get_media_entry_by_id
@require_active_login
-def media_confirm_delete(request):
-
- allowed_state = [u'failed', u'processed']
- media = None
- for media_state in allowed_state:
- media = request.db.MediaEntry.query.filter_by(id=request.matchdict['media_id'], state=media_state).first()
- if media:
- break
-
- if not media:
- return render_404(request)
-
- given_username = request.matchdict.get('user')
- if given_username and (given_username != media.get_uploader.username):
- return render_404(request)
-
- uploader_id = media.uploader
- if not (request.user.has_privilege(u'admin') or
- request.user.id == uploader_id):
- raise Forbidden()
-
+@user_may_delete_media
+def media_confirm_delete(request, media):
+
form = user_forms.ConfirmDeleteForm(request.form)
if request.method == 'POST' and form.validate():