Can't delete draft blog posts

[ayleph]

Draft blog posts are set to the 'failed' state. The generated blogpost_delete_url link for a draft gets routed to a 404 page.

[ayleph]

And here's why. First, the deletion URL is created using the media_confirm_delete generator.

blogpost_delete_url = request.urlgen('mediagoblin.user_pages.media_confirm_delete',                                 
              user= blog_post.get_actor.username,
              media_id=blog_post.id)

The media_confirm_delete view is decorated by @get_media_entry_by_id.

@get_media_entry_by_id
@require_active_login
@user_may_delete_media
def media_confirm_delete(request, media):               

The @get_media_entry_by_id generator returns a 404 for media that aren't in the processed state.

def get_media_entry_by_id(controller):
    """
    Pass in a MediaEntry based off of a url component
    """
    @wraps(controller)
    def wrapper(request, *args, **kwargs):
        media = MediaEntry.query.filter_by(
                id=request.matchdict['media_id'],
                state=u'processed').first()
        # Still no media?  Okay, 404.
        if not media:
            return render_404(request)

Since blog post drafts aren't processed, the delete URL returns a 404.

[ayleph]

I've attached a patch which I think addresses this issue. I removed the requirement from @get_media_entry_by_id for the media to be processed. This allows unprocessed media to be deleted. For all of the other views which used the decorator, I added constraints that the media must be processed.

[ayleph]

I've been using this patch for weeks without issue. Pushed to master in ​2f2b4cb.