Changeset 27767 for trunk/src/wp-includes/post.php

Timestamp:

03/27/2014 01:17:41 AM (11 years ago)

Author:

wonderboymusic

Message:

Use wp_parse_id_list() when parsing exclude_tree in get_pages(). Add unit tests to ensure a URL string, array with string as value, and array with array as value for exclude_tree can be used to specify multiple IDs.

Props cgaffga, roothorick, hakre, tbrams for patches across the years.
Fixes #9153.

File:

• trunk/src/wp-includes/post.php (modified) (2 diffs)

trunk/src/wp-includes/post.php

@@ -3743 +3743 @@
         'exclude' => array(), 'include' => array(),
         'meta_key' => '', 'meta_value' => '',
-        'authors' => '', 'parent' => -1, 'exclude_tree' => '',
+        'authors' => '', 'parent' => -1, 'exclude_tree' => array(),
         'number' => '', 'offset' => 0,
         'post_type' => 'page', 'post_status' => 'publish',
@@ -3925 +3925 @@
         $pages = get_page_children($child_of, $pages);
 
-    if ( !empty($exclude_tree) ) {
-        $exclude = (int) $exclude_tree;
-        $children = get_page_children($exclude, $pages);
-        $excludes = array();
-        foreach ( $children as $child )
-            $excludes[] = $child->ID;
-        $excludes[] = $exclude;
-        $num_pages = count($pages);
+    if ( ! empty( $exclude_tree ) ) {
+        $exclude = wp_parse_id_list( $exclude_tree );
+        foreach( $exclude as $id ) {
+            $children = get_page_children( $id, $pages );
+            foreach ( $children as $child ) {
+                $exclude[] = $child->ID;
+            }
+        }
+
+        $num_pages = count( $pages );
         for ( $i = 0; $i < $num_pages; $i++ ) {
-            if ( in_array($pages[$i]->ID, $excludes) )
-                unset($pages[$i]);
+            if ( in_array( $pages[$i]->ID, $exclude ) ) {
+                unset( $pages[$i] );
+            }
         }
     }