Changeset 12304

Timestamp:

12/03/2018 08:53:09 PM (8 years ago)

Author:

boonebgorges

Message:

Introduce bp_is_large_install() and use throughout BuddyPress.

This checks provides a fallback that is not dependent on Multisite, and
helps prevent a fatal error introduced in [12241]. See #7980.

Merges [12303] to the 4.0 branch.

Props shanebp, dcavins, johnjamesjacoby.
Fixes #8012.

Location:

branches/4.0

Files:

• . (modified) (1 prop)
• src/bp-blogs/bp-blogs-functions.php (modified) (1 diff)
• src/bp-core/bp-core-functions.php (modified) (1 diff)
• src/bp-friends/bp-friends-functions.php (modified) (1 diff)
• src/bp-groups/bp-groups-admin.php (modified) (2 diffs)

branches/4.0/src/bp-blogs/bp-blogs-functions.php

@@ -130 +130 @@
 
                 // Omit root blog if large network
-                if ( wp_is_large_network( 'users' ) ) {
+                if ( bp_is_large_install() ) {
                         $sql['omit_root_blog'] = $wpdb->prepare( "AND blog_id != %d", bp_get_root_blog_id() );
                 }

branches/4.0/src/bp-core/bp-core-functions.php

@@ -3901 +3901 @@
         return preg_replace( '@<(script|style)[^>]*?>.*?</\\1>@si', '', $string );
 }
+
+/**
+ * Checks whether the current installation is "large".
+ *
+ * By default, an installation counts as "large" if there are 10000 users or more.
+ * Filter 'bp_is_large_install' to adjust.
+ *
+ * @since 4.1.0
+ *
+ * @return bool
+ */
+function bp_is_large_install() {
+        // Use the Multisite function if available.
+        if ( function_exists( 'wp_is_large_network' ) ) {
+                $is_large = wp_is_large_network( 'users' );
+        } else {
+                $is_large = bp_core_get_total_member_count() > 10000;
+        }
+
+        /**
+         * Filters whether the current installation is "large".
+         *
+         * @since 4.1.0
+         *
+         * @param bool $is_large True if the network is "large".
+         */
+        return (bool) apply_filters( 'bp_is_large_install', $is_large );
+}

branches/4.0/src/bp-friends/bp-friends-functions.php

@@ -782 +782 @@
 
         // Bail out if the site has a ton of users.
-        if ( is_multisite() && wp_is_large_network( 'users' ) ) {
+        if ( bp_is_large_install() ) {
                 return;
         }

branches/4.0/src/bp-groups/bp-groups-admin.php

@@ -862 +862 @@
  */
 function bp_groups_admin_edit_metabox_add_new_members( $item ) {
-        if ( wp_is_large_network( 'users' ) ) {
+        if ( bp_is_large_install() ) {
                 $class  = '';
                 $notice = __( 'Enter a comma-separated list of user logins.', 'buddypress' );
@@ -1253 +1253 @@
 
         // Bail if user user shouldn't be here, or is a large network.
-        if ( ! bp_current_user_can( 'bp_moderate' ) || ( is_multisite() && wp_is_large_network( 'users' ) ) ) {
+        if ( ! bp_current_user_can( 'bp_moderate' ) || bp_is_large_install() ) {
                 wp_die( -1 );
         }