Changeset 8584

Timestamp:

07/10/2014 07:57:44 PM (10 years ago)

Author:

johnjamesjacoby

Message:

Check for wp_is_large_network() before looping through all blogs in the network, to prevent looping through thousands of sites on large networks. If you have this many sites, you should manage this in a queue or job system. See #5749.

File:

• trunk/src/bp-blogs/bp-blogs-functions.php (modified) (1 diff)

trunk/src/bp-blogs/bp-blogs-functions.php

@@ -75 +75 @@
     $wpdb->query( "DELETE FROM {$bp->blogs->table_name} WHERE 1=1" );
 
+    // Query for all sites in network
     if ( is_multisite() ) {
-        $blog_ids = $wpdb->get_col( $wpdb->prepare( "SELECT blog_id FROM {$wpdb->base_prefix}blogs WHERE mature = 0 AND spam = 0 AND deleted = 0 AND site_id = %d", $wpdb->siteid ) );
+
+        // Get blog ID's if not a large network
+        if ( ! wp_is_large_network() ) {
+            $blog_ids = $wpdb->get_col( $wpdb->prepare( "SELECT blog_id FROM {$wpdb->base_prefix}blogs WHERE mature = 0 AND spam = 0 AND deleted = 0 AND site_id = %d", $wpdb->siteid ) );
+
+        // Large networks are not currently supported
+        } else {
+            $blog_ids = false;
+        }
+
+    // Record a single site
     } else {
         $blog_ids = 1;