Use media.url_for_self() instead of generating it

Some places used to generate the URL for a media entry on
their own instead of calling media.url_for_self() to do
that. The later handles missing slugs better.

1 files changed, 4 insertions, 5 deletions

diff --git a/mediagoblin/edit/views.py b/mediagoblin/edit/views.py
index 11bee110..15edfdd6 100644
--- a/mediagoblin/edit/views.py
+++ b/mediagoblin/edit/views.py
@@ -73,8 +73,8 @@ def edit_media(request, media):
             media['slug'] = unicode(request.POST['slug'])
             media.save()
 
-            return redirect(request, "mediagoblin.user_pages.media_home",
-                user=media.uploader()['username'], media=media['slug'])
+            return exc.HTTPFound(
+                location=media.url_for_self(request.urlgen))
 
     if request.user['is_admin'] \
             and media['uploader'] != request.user['_id'] \
@@ -130,9 +130,8 @@ def edit_attachments(request, media):
                     % (request.POST['attachment_name']
                        or request.POST['attachment_file'].filename))
 
-            return redirect(request, 'mediagoblin.user_pages.media_home',
-                            user=media.uploader()['username'],
-                            media=media['slug'])
+            return exc.HTTPFound(
+                location=media.url_for_self(request.urlgen))
         return render_to_response(
             request,
             'mediagoblin/edit/attachments.html',