diff options
author | Sebastian Spaeth <Sebastian@SSpaeth.de> | 2013-12-12 17:38:18 +0100 |
---|---|---|
committer | Sebastian Spaeth <Sebastian@SSpaeth.de> | 2013-12-12 17:38:18 +0100 |
commit | 686cbcd974b464839b09338bab2bc11040c1043b (patch) | |
tree | 135a0b3207d0bf86a58d478dda8d054b85f62940 | |
parent | 52a0387fc6ad023516c765019725ca58daaae347 (diff) | |
download | mediagoblin-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.py | 26 |
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(): |