[wp-trac] [WordPress Trac] #12959: New menu systems unrecoverable error

WordPress Trac wp-trac at lists.automattic.com
Sun Apr 11 15:56:32 UTC 2010


#12959: New menu systems unrecoverable error
--------------------------+-------------------------------------------------
 Reporter:  Dickie        |       Owner:  ryan          
     Type:  defect (bug)  |      Status:  new           
 Priority:  high          |   Milestone:  3.0           
Component:  Menus         |     Version:  3.0           
 Severity:  major         |    Keywords:  menu has-patch
--------------------------+-------------------------------------------------
Changes (by nacin):

  * keywords:  menu has-patch commit => menu has-patch
  * milestone:  Unassigned => 3.0


Comment:

 If a post gets deleted and a menu item is a term with the same ID as the
 post, it accidentally gets the axe.

 We don't account for trashed posts. I've additionally moved the actions to
 the post-delete actions (deleted_post versus delete_post). And we need to
 call wp_delete_post with a $force_delete arg.

 Wouldn't this cause a potential loop? i.e. if the function needs to delete
 a nav_menu_item post, then it'll fire all the hooks all over again for
 that. We need a way to stop that.

 Posting my patch so far.

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


More information about the wp-trac mailing list