heckyel/mediagoblin
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Fix some media page redirects in media_collect.

Browse Source 
Use .slug_or_id instead of only .id.
This commit is contained in:
Elrond 2013-03-01 14:45:20 +01:00
parent 918d811a78
commit b7a3798e18
1 changed files with 3 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
mediagoblin/user_pages/views.py
View File

@ -214,8 +214,8 @@ def media_collect(request, media):
_('You already have a collection called "%s"!') _('You already have a collection called "%s"!')
% existing_collection.title) % existing_collection.title)
return redirect(request, "mediagoblin.user_pages.media_home", return redirect(request, "mediagoblin.user_pages.media_home",
user=request.user.username, user=media.get_uploader.username,
media=media.id) media=media.slug_or_id)
collection = Collection() collection = Collection()
collection.title = request.form['collection_title'] collection.title = request.form['collection_title']
@ -266,7 +266,7 @@ def media_collect(request, media):
return redirect(request, "mediagoblin.user_pages.media_home", return redirect(request, "mediagoblin.user_pages.media_home",
user=media.get_uploader.username, user=media.get_uploader.username,
media=media.id) media=media.slug_or_id)
#TODO: Why does @user_may_delete_media not implicate @require_active_login? #TODO: Why does @user_may_delete_media not implicate @require_active_login?