[wp-trac] [WordPress Trac] #11018: display_page_row loops forever

WordPress Trac wp-trac at lists.automattic.com
Sat Oct 24 21:19:54 UTC 2009


#11018: display_page_row loops forever
--------------------------+-------------------------------------------------
 Reporter:  hailin        |       Owner:  hailin     
     Type:  defect (bug)  |      Status:  new        
 Priority:  normal        |   Milestone:  2.9        
Component:  Template      |     Version:  2.7        
 Severity:  normal        |    Keywords:  needs-patch
--------------------------+-------------------------------------------------
Changes (by westi):

  * keywords:  has-patch => needs-patch


Comment:

 I'm not sure this patch is the best fix for the overall issue.

 I think we need to approach this from two angles:

  * Stopping it happening by validating the parent is valid when the
 request to set it happens.
  * Ensure if it still happens we recover in a meaningful way.

 Also it looks like the code patched here should use get_post_ancestors
 which would have already built the tree for us and we can just
 {{{count()}}} it.

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


More information about the wp-trac mailing list