[wp-trac] [WordPress Trac] #18402: Confused page ordering if filter drops parent pages

[WordPress Trac]

#18402: Confused page ordering if filter drops parent pages
-----------------------------------------------+---------------------------
 Reporter:  erdnah                             |       Owner:  nacin
     Type:  defect (bug)                       |      Status:  reviewing
 Priority:  normal                             |   Milestone:  Awaiting
Component:  Administration                     |  Review
 Severity:  normal                             |     Version:  3.2.1
 Keywords:  has-patch 2nd-opinion ui-feedback  |  Resolution:
-----------------------------------------------+---------------------------
Changes (by pdclark):

 * keywords:  has-patch 2nd-opinion => has-patch 2nd-opinion ui-feedback


Comment:

 Replying to [comment:6 MikeHansenMe]:
 > Tested on 3.5-alpha-21751 and it works as described. My only question
 would be should "grandchild" then show "child" as the parent? ex
 (Grandchild | Parent Page : Child)

 The [https://github.com/WordPress/WordPress/blob/master/wp-admin/includes
 /class-wp-posts-list-table.php#L525 code that adds the filter] for "Parent
 Page:" to display is commented:
 {{{
 //sent level 0 by accident, by default, or because we don't know the
 actual level
 }}}

 Because this patch causes the child and grandchild page's hierarchical
 level to be set correctly (not sent in error or unknown), "Parent Page:"
 doesn't display for either the child or the grandchild. See the furthest-
 right window in
 [http://core.trac.wordpress.org/attachment/ticket/18402/21272-trashing-
 parents.jpg this screenshot]

 Who should make the final call on this?

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