sergiotarxz/Wordpress
1
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

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

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

Props Chouby.
Fixes #50463.

git-svn-id: https://develop.svn.wordpress.org/trunk@48282 602fd350-edb4-49c9-b593-d223f7449a82
This commit is contained in:
Pascal Birchler 2020-07-03 12:01:46 +00:00
parent 950b6ed272
commit 65a6412681
1 changed files with 1 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
src/wp-includes/sitemaps/providers/class-wp-sitemaps-posts.php
View File

@ -92,13 +92,6 @@ class WP_Sitemaps_Posts extends WP_Sitemaps_Provider {
$query = new WP_Query( $args );
/**
* Returns an array of posts.
*
* @var array<int, \WP_Post> $posts
*/
$posts = $query->get_posts();
$url_list = array();
/*
@ -122,7 +115,7 @@ class WP_Sitemaps_Posts extends WP_Sitemaps_Provider {
$url_list[] = $sitemap_entry;
}
foreach ( $posts as $post ) {
foreach ( $query->posts as $post ) {
$sitemap_entry = array(
'loc' => get_permalink( $post ),
);