Ticket #38741: 38741.3.diff

src/wp-admin/includes/class-wp-users-list-table.php

@@ -173 +173 @@
                 global $role;
 
                 $wp_roles = wp_roles();
+                $count_users = true;
 
+                if ( wp_is_large_site() ) {
+                        $count_users = false;
+                } elseif ( is_multisite() && wp_is_large_network( 'users' ) ) {
+                        $count_users = false;
+                }
+
                 if ( $this->is_site_users ) {
                         $url = 'site-users.php?id=' . $this->site_id;
-                        switch_to_blog( $this->site_id );
-                        $users_of_blog = count_users();
-                        restore_current_blog();
+                        if ( $count_users ) {
+                                switch_to_blog( $this->site_id );
+                                $users_of_blog = count_users();
+                                restore_current_blog();
+                        }
                 } else {
                         $url = 'users.php';
-                        $users_of_blog = count_users();
+                        if ( $count_users ) {
+                                $users_of_blog = count_users();
+                        }
                 }
 
-                $total_users = $users_of_blog['total_users'];
-                $avail_roles =& $users_of_blog['avail_roles'];
-                unset($users_of_blog);
+                if ( $count_users ) {
+                        $total_users =  $users_of_blog['total_users'];
+                        $avail_roles =& $users_of_blog['avail_roles'];
+                        unset( $users_of_blog );
+                } else {
+                        $avail_roles = array();
+                }
 
                 $class = empty($role) ? ' class="current"' : '';
                 $role_links = array();
-                $role_links['all'] = "<a href='$url'$class>" . sprintf( _nx( 'All <span class="count">(%s)</span>', 'All <span class="count">(%s)</span>', $total_users, 'users' ), number_format_i18n( $total_users ) ) . '</a>';
+
+                if ( $count_users ) {
+                        $role_links['all'] = "<a href='$url'$class>" . sprintf( _nx( 'All <span class="count">(%s)</span>', 'All <span class="count">(%s)</span>', $total_users, 'users' ), number_format_i18n( $total_users ) ) . '</a>';
+                } else {
+                        $role_links['all'] = "<a href='$url'$class>" . _x( 'All', 'users' ) . '</a>';
+                }
+
                 foreach ( $wp_roles->get_names() as $this_role => $name ) {
-                        if ( !isset($avail_roles[$this_role]) )
+                        if ( $count_users && ! isset( $avail_roles[ $this_role ] ) ) {
                                 continue;
+                        }
 
                         $class = '';
 
@@ -202 +224 @@
                         }
 
                         $name = translate_user_role( $name );
-                        /* translators: User role name with count */
-                        $name = sprintf( __('%1$s <span class="count">(%2$s)</span>'), $name, number_format_i18n( $avail_roles[$this_role] ) );
+                        if ( $count_users ) {
+                                /* translators: User role name with count */
+                                $name = sprintf( __('%1$s <span class="count">(%2$s)</span>'), $name, number_format_i18n( $avail_roles[$this_role] ) );
+                        }
                         $role_links[$this_role] = "<a href='" . esc_url( add_query_arg( 'role', $this_role, $url ) ) . "'$class>$name</a>";
                 }
 
-                if ( ! empty( $avail_roles['none' ] ) ) {
+                if ( ! $count_users || ! empty( $avail_roles['none' ] ) ) {
 
                         $class = '';
 
@@ -216 +240 @@
                         }
 
                         $name = __( 'No role' );
-                        /* translators: User role name with count */
-                        $name = sprintf( __('%1$s <span class="count">(%2$s)</span>'), $name, number_format_i18n( $avail_roles['none' ] ) );
+                        if ( $count_users ) {
+                                /* translators: User role name with count */
+                                $name = sprintf( __('%1$s <span class="count">(%2$s)</span>'), $name, number_format_i18n( $avail_roles['none' ] ) );
+                        }
                         $role_links['none'] = "<a href='" . esc_url( add_query_arg( 'role', 'none', $url ) ) . "'$class>$name</a>";
 
                 }

src/wp-includes/functions.php

@@ -5642 +5642 @@
 
         return $last_changed;
 }
+
+/**
+ * Whether or not we have a large site.
+ *
+ * The default criteria for a large site is more than 10,000 users.
+ *
+ * @since x.x.x
+ *
+ * @return bool True if the site meets the criteria for large. False otherwise.
+ */
+function wp_is_large_site() {
+        $count = wp_get_active_user_count();
+        /**
+         * Filters whether the site is considered large.
+         *
+         * @since x.x.x
+         *
+         * @param bool   $is_large_site Whether the site has more than 10000 users.
+         * @param int    $count         The count of items for the component.
+         */
+        return apply_filters( 'wp_is_large_site', $count > 10000, $count );
+}
+
+/**
+ * Update the active user count.
+ *
+ * @since x.x.x
+ *
+ * @global wpdb $wpdb WordPress database abstraction object.
+ */
+function wp_active_user_count() {
+        global $wpdb;
+
+        $count = $wpdb->get_var( "SELECT COUNT(ID) as c FROM $wpdb->users" );
+        return $count;
+}
+
+/**
+ * The number of active users.
+ *
+ * The count is cached for a minimum of twelve hours. This is not a live count.
+ *
+ * @since x.x.x
+ *
+ * @return int
+ */
+function wp_get_active_user_count() {
+        $count = get_transient( 'active_user_count' );
+
+        if ( false === $count ) {
+                $count = wp_active_user_count();
+                set_transient( 'active_user_count', $count, 12 * HOUR_IN_SECONDS );
+        }
+
+        return (int) $count;
+}

src/wp-includes/update.php

@@ -78 +78 @@
                 $wp_install = network_site_url();
                 $multisite_enabled = 1;
         } else {
-                $user_count = count_users();
-                $user_count = $user_count['total_users'];
+                $user_count = wp_get_active_user_count();
                 $multisite_enabled = 0;
                 $num_blogs = 1;
                 $wp_install = home_url( '/' );

tests/phpunit/tests/functions.php

@@ -1114 +1114 @@
 
                 return $data;
         }
+
+        /*
+         * @ticket 38741
+         */
+        
+        public function test_wp_get_active_user_count(){
+
+                // make twenty additional users to make 21 users
+                for( $i = 0; $i < 20; $i++ ){
+                        self::factory()->user->create();
+                }
+
+                $user_count = wp_active_user_count();
+
+                $this->assertEquals( $user_count, 21 );
+        }
+
+        /*
+         * @ticket 38741
+         */
+        public function test_wp_is_large_site(){
+
+                if ( wp_using_ext_object_cache() ) {
+                        $this->markTestSkipped( 'Not testable with an external object cache.' );
+                }
+
+                // set the 'active_user_count' transient to over 10000 to emulate a large site
+                set_transient( 'active_user_count', 10001 );
+                $this->assertTrue( wp_is_large_site() );
+                delete_transient( 'active_user_count' );
+        }
 }