diff options
| author | Elrond <elrond+mediagoblin.org@samba-tng.org> | 2013-01-11 14:18:27 +0100 |
|---|---|---|
| committer | Elrond <elrond+mediagoblin.org@samba-tng.org> | 2013-01-11 21:48:03 +0100 |
| commit | 461dd9717cce6c5b4d40bb4e76ca65d9d898d1df (patch) | |
| tree | 9b55136a4c5353ed3e65dc6c82f7ecad7d2cd784 /mediagoblin/edit | |
| parent | 8b271c28bd4c0b9c90990d87c1dfd0d8d95b9e45 (diff) | |
| download | mediagoblin-461dd9717cce6c5b4d40bb4e76ca65d9d898d1df.tar.lz mediagoblin-461dd9717cce6c5b4d40bb4e76ca65d9d898d1df.tar.xz mediagoblin-461dd9717cce6c5b4d40bb4e76ca65d9d898d1df.zip | |
Start to use the media_id in "admin" URLs.
We have a bunch of URLs that are more for internal use. At
least they're definitely not intended to be posted
somewhere for long term useage.
When those things affect a media, it's much better to
reference the media by its id. This can't change, ever.
This is better for races.
Like someone posting a comment while the owner
corrects a typo in the slug.
Diffstat (limited to 'mediagoblin/edit')
| -rw-r--r-- | mediagoblin/edit/views.py | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/mediagoblin/edit/views.py b/mediagoblin/edit/views.py index 2f669c66..505106a4 100644 --- a/mediagoblin/edit/views.py +++ b/mediagoblin/edit/views.py @@ -27,6 +27,7 @@ from mediagoblin.auth import lib as auth_lib from mediagoblin.edit import forms from mediagoblin.edit.lib import may_edit_media from mediagoblin.decorators import (require_active_login, active_user_from_url, + get_media_entry_by_id, get_user_media_entry, user_may_alter_collection, get_user_collection) from mediagoblin.tools.response import render_to_response, redirect from mediagoblin.tools.translate import pass_to_ugettext as _ @@ -37,7 +38,7 @@ from mediagoblin.db.util import check_media_slug_used, check_collection_slug_use import mimetypes -@get_user_media_entry +@get_media_entry_by_id @require_active_login def edit_media(request, media): if not may_edit_media(request, media): |