Changeset 48282

Timestamp:

07/03/2020 12:01:46 PM (6 years ago)

Author:

swissspidy

Message:

Sitemaps: Do not unnecessarily call WP_Query::get_posts() in posts provider.

The posts have already been fetched at this point, no need to do it again.

Props Chouby.
Fixes #50463.

File:

• trunk/src/wp-includes/sitemaps/providers/class-wp-sitemaps-posts.php (modified) (2 diffs)

trunk/src/wp-includes/sitemaps/providers/class-wp-sitemaps-posts.php

@@ -93 +93 @@
                 $query = new WP_Query( $args );
 
-                /**
-                 * Returns an array of posts.
-                 *
-                 * @var array<int, \WP_Post> $posts
-                 */
-                $posts = $query->get_posts();
-
                 $url_list = array();
 
@@ -123 +116 @@
                 }
 
-                foreach ( $posts as $post ) {
+                foreach ( $query->posts as $post ) {
                         $sitemap_entry = array(
                                 'loc' => get_permalink( $post ),