Leave slug empty until we are sure media processing was successful.

Patch submitted by LotusEcho

mediagoblin/plugins/api/views.py

@@ -61,8 +61,6 @@ def post_entry(request):
     entry.description = unicode(request.form.get('description'))
     entry.license = unicode(request.form.get('license', ''))
 
-    entry.generate_slug()
-
     # queue appropriately
     queue_file = prepare_queue_task(request.app, entry, media_file.filename)
 

mediagoblin/processing/task.py

@@ -94,6 +94,8 @@ class ProcessMedia(task.Task):
             # We set the state to processed and save the entry here so there's
             # no need to save at the end of the processing stage, probably ;)
             entry.state = u'processed'
+            #Generate the slug here rather than earlier when it could have failed.
+            entry.generate_slug()
             entry.save()
 
             # Notify the PuSH servers as async task

mediagoblin/submit/views.py

@@ -78,9 +78,6 @@ def submit_start(request):
                 entry.tags = convert_to_tag_list_of_dicts(
                     submit_form.tags.data)
 
-                # Generate a slug from the title
-                entry.generate_slug()
-
                 queue_file = prepare_queue_task(request.app, entry, filename)
 
                 with queue_file: