From: Elrond <elrond+mediagoblin.org@samba-tng.org>
Date: Fri, 1 Mar 2013 13:45:20 +0000 (+0100)
Subject: Fix some media page redirects in media_collect.
X-Git-Url: https://vcs.fsf.org/?a=commitdiff_plain;h=b7a3798e1806eb52a244088fb600376ff25e0de9;p=mediagoblin.git

Fix some media page redirects in media_collect.

Use .slug_or_id instead of only .id.
---

diff --git a/mediagoblin/user_pages/views.py b/mediagoblin/user_pages/views.py
index 6f8f8059..69d7defb 100644
--- a/mediagoblin/user_pages/views.py
+++ b/mediagoblin/user_pages/views.py
@@ -214,8 +214,8 @@ def media_collect(request, media):
                 _('You already have a collection called "%s"!')
                 % existing_collection.title)
             return redirect(request, "mediagoblin.user_pages.media_home",
-                            user=request.user.username,
-                            media=media.id)
+                            user=media.get_uploader.username,
+                            media=media.slug_or_id)
 
         collection = Collection()
         collection.title = request.form['collection_title']
@@ -266,7 +266,7 @@ def media_collect(request, media):
 
     return redirect(request, "mediagoblin.user_pages.media_home",
                     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?