Changeset 60632

Timestamp:

08/13/2025 04:26:33 PM (8 months ago)

Author:

SergeyBiryukov

Message:

Export: Split the query for post authors in wxr_authors_list() into smaller chunks.

This aims to avoid a fatal error when attempting to export content on larger WP instances with a lot of data.

Follow-up to [15961], [28731].

Props bor0, SirLouen, SergeyBiryukov.
Fixes #63503.

File:

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

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

@@ -406 +406 @@
 
         if ( ! empty( $post_ids ) ) {
-            $post_ids = array_map( 'absint', $post_ids );
-            $and      = 'AND ID IN ( ' . implode( ', ', $post_ids ) . ')';
+            $post_ids       = array_map( 'absint', $post_ids );
+            $post_id_chunks = array_chunk( $post_ids, 20 );
         } else {
-            $and = '';
+            $post_id_chunks = array( array() );
         }
 
         $authors = array();
-        $results = $wpdb->get_results( "SELECT DISTINCT post_author FROM $wpdb->posts WHERE post_status != 'auto-draft' $and" );
-        foreach ( (array) $results as $result ) {
-            $authors[] = get_userdata( $result->post_author );
+
+        foreach ( $post_id_chunks as $next_posts ) {
+            $and = ! empty( $next_posts ) ? 'AND ID IN (' . implode( ', ', $next_posts ) . ')' : '';
+
+            $results = $wpdb->get_results( "SELECT DISTINCT post_author FROM $wpdb->posts WHERE post_status != 'auto-draft' $and" );
+
+            foreach ( (array) $results as $result ) {
+                $authors[] = get_userdata( $result->post_author );
+            }
         }
 
         $authors = array_filter( $authors );
+        $authors = array_unique( $authors, SORT_REGULAR ); // Remove duplicate authors.
 
         foreach ( $authors as $author ) {