Changeset 44338

Timestamp:

12/19/2018 10:22:50 PM (4 years ago)

Author:

desrosj

Message:

Posts, Post Types: Correctly show hierarchical post type hierarchy in admin.

In [44185], a bug was introduced where hierarchical post types would not display in the correct default order (hierarchically).

This was caused by a ! isset() check, which returned false after [44185], causing the correct default value to not be applied. This switches that conditional to use an empty() check, ignoring the new empty string assignment that was added to prevent a PHP notice when compact() is called.

Props davidbinda.
Fixes #45711.

File:

• trunk/src/wp-admin/includes/post.php (modified) (1 diff)

trunk/src/wp-admin/includes/post.php

@@ -1158 +1158 @@
 
     // Hierarchical types require special args.
-    if ( is_post_type_hierarchical( $post_type ) && ! isset( $orderby ) ) {
+    if ( is_post_type_hierarchical( $post_type ) && empty( $orderby ) ) {
         $query['orderby']                = 'menu_order title';
         $query['order']                  = 'asc';