[wp-trac] [WordPress Trac] #11534: WordPress moves post/page to trash instead of deleting it

WordPress Trac wp-trac at lists.automattic.com
Sun Dec 20 19:59:07 UTC 2009


#11534: WordPress moves post/page to trash instead of deleting it
--------------------------+-------------------------------------------------
 Reporter:  sirzooro      |       Owner:           
     Type:  defect (bug)  |      Status:  new      
 Priority:  normal        |   Milestone:  2.9.1    
Component:  Trash         |     Version:  2.9      
 Severity:  normal        |    Keywords:  has-patch
--------------------------+-------------------------------------------------
Changes (by sirzooro):

  * keywords:  needs-patch => has-patch


Comment:

 Replying to [comment:3 nacin]:
 > That said, action=delete links should be processed always as a force
 delete, not as a conditional for whether trash is enabled (note how $force
 = !EMPTY_TRASH_DAYS). There's a reason we have action=trash as well.

 I agree with this, and my patch works in this way.

 As I checked, for now I can hook actions trashed_post/trashed_comment,
 check if `$_GET['action']=='delete'` and delete again (this time
 post/page/comment will be in trash). This works, but it is an ugly hack -
 I do not like it.

 BTW, I have to check how bulk actions work - I plan to add bulk delete
 option too.

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


More information about the wp-trac mailing list