[wp-trac] [WordPress Trac] #11047: admin edit pages could loop

WordPress Trac wp-trac at lists.automattic.com
Fri Oct 30 17:08:41 UTC 2009


#11047: admin edit pages could loop
--------------------------+-------------------------------------------------
 Reporter:  hailin        |       Owner:     
     Type:  defect (bug)  |      Status:  new
 Priority:  normal        |   Milestone:  2.9
Component:  General       |     Version:  2.7
 Severity:  normal        |    Keywords:     
--------------------------+-------------------------------------------------

Comment(by hailin):

 Attached the right patch.

 _get_post_ancestors has the following comment:
 The post parent will be an ancestor and the parent of the post
  * parent will be an ancestor. There will only be two ancestors at the
 most.

 Based on the above comment, _get_post_ancestors doesn't have loop issue,
 and it appears that it was designed to have only two ancestors.

 Therefore, the correct way to break pages loop is in display_page_row.

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


More information about the wp-trac mailing list