Opened 6 years ago
Closed 6 years ago
#5513 closed defect (fixed)
Can't delete draft blog posts
|Reported by:||ayleph||Owned by:|
Draft blog posts are set to the 'failed' state. The generated blogpost_delete_url link for a draft gets routed to a 404 page.
Change History (7)
comment:1 by , 6 years ago
by , 6 years ago
comment:2 by , 6 years ago
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.
comment:3 by , 6 years ago
|Status:||new → review|
comment:4 by , 6 years ago
comment:5 by , 6 years ago
comment:6 by , 6 years ago
|Status:||review → closed|
I've been using this patch for weeks without issue. Pushed to master in 2f2b4cb.
And here's why. First, the deletion URL is created using the
media_confirm_deleteview is decorated by
@get_media_entry_by_idgenerator returns a 404 for media that aren't in the processed state.
Since blog post drafts aren't processed, the delete URL returns a 404.