diff options
| author | Elrond <elrond+mediagoblin.org@samba-tng.org> | 2013-03-01 14:45:20 +0100 |
|---|---|---|
| committer | Elrond <elrond+mediagoblin.org@samba-tng.org> | 2013-03-01 22:08:24 +0100 |
| commit | b7a3798e1806eb52a244088fb600376ff25e0de9 (patch) | |
| tree | b561012df404536c8e941fe384412e3ce101f512 | |
| parent | 918d811a78839ba156a88a15dd115ce924ea5553 (diff) | |
| download | mediagoblin-b7a3798e1806eb52a244088fb600376ff25e0de9.tar.lz mediagoblin-b7a3798e1806eb52a244088fb600376ff25e0de9.tar.xz mediagoblin-b7a3798e1806eb52a244088fb600376ff25e0de9.zip | |
Fix some media page redirects in media_collect.
Use .slug_or_id instead of only .id.
| -rw-r--r-- | mediagoblin/user_pages/views.py | 6 |
1 files changed, 3 insertions, 3 deletions
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? |