Changeset 55444 for trunk/src/wp-includes/author-template.php

Timestamp:

03/01/2023 03:37:06 PM (19 months ago)

Author:

SergeyBiryukov

Message:

Users: Use a separate variable for the post counts query in wp_list_authors().

This avoids a collision if wp_list_authors() is called with the optioncount parameter enabled, and the resulting array for the post counts query contains a key that matches the ID of a user who does not have any posts.

Follow-up to [54262].

Props peterwilsoncc, johnbillion, lifeboat, brookedot, thedaysse, Toru.
Fixes #57011.

File:

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

trunk/src/wp-includes/author-template.php

@@ -482 +482 @@
 
     if ( ! is_array( $post_counts ) ) {
-        $post_counts = $wpdb->get_results(
+        $post_counts       = array();
+        $post_counts_query = $wpdb->get_results(
             "SELECT DISTINCT post_author, COUNT(ID) AS count
             FROM $wpdb->posts
@@ -489 +490 @@
         );
 
-        foreach ( (array) $post_counts as $row ) {
+        foreach ( (array) $post_counts_query as $row ) {
             $post_counts[ $row->post_author ] = $row->count;
         }