[wp-trac] [WordPress Trac] #21194: Delete attachment AJAX fails when EMPTY_TRASH_DAYS is set to 0 days

[WordPress Trac]

#21194: Delete attachment AJAX fails when EMPTY_TRASH_DAYS is set to 0 days
---------------------------+------------------------------
 Reporter:  simonwheatley  |       Owner:
     Type:  defect (bug)   |      Status:  new
 Priority:  normal         |   Milestone:  Awaiting Review
Component:  Media          |     Version:  2.0
 Severity:  normal         |  Resolution:
 Keywords:  dev-feedback   |
---------------------------+------------------------------
Changes (by c3mdigital):

 * version:   => 2.0


Comment:

 It also fails on move to trash when MEDIA_TRASH is set to true.

 Do we want to do a check for EMPTY_TRASH_DAYS on attachments since the
 only difference currently is "delete immediately" vs the "confirm
 deletion" routine?

 The attached patch removes the check for EMPTY_TRASH_DAYS and removes
 class="delete" which fixes the fail on moving to trash.  Not sure if it
 would be better to remove the AJAX binding on a.delete.

-- 
Ticket URL: <http://core.trac.wordpress.org/ticket/21194#comment:2>
WordPress Trac <http://core.trac.wordpress.org/>
WordPress blogging software