From 645a5501293e3c6771d568b01781d12515c7bf10 Mon Sep 17 00:00:00 2001 From: John Blackbourn Date: Wed, 4 Oct 2017 22:16:21 +0000 Subject: [PATCH] Users: Revert [41613], [41614], and [41623] as this feature needs some more work. See #38741 git-svn-id: https://develop.svn.wordpress.org/trunk@41753 602fd350-edb4-49c9-b593-d223f7449a82 --- .../includes/class-wp-users-list-table.php | 53 +++++----------- src/wp-includes/default-filters.php | 5 -- src/wp-includes/functions.php | 61 ------------------- src/wp-includes/ms-functions.php | 7 +-- src/wp-includes/update.php | 3 +- 5 files changed, 17 insertions(+), 112 deletions(-) diff --git a/src/wp-admin/includes/class-wp-users-list-table.php b/src/wp-admin/includes/class-wp-users-list-table.php index 7143d94b3c..7382f4de38 100644 --- a/src/wp-admin/includes/class-wp-users-list-table.php +++ b/src/wp-admin/includes/class-wp-users-list-table.php @@ -166,49 +166,28 @@ class WP_Users_List_Table extends WP_List_Table { global $role; $wp_roles = wp_roles(); - $count_users = true; - - if ( wp_is_large_user_count() ) { - $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; - if ( $count_users ) { - switch_to_blog( $this->site_id ); - $users_of_blog = count_users( 'time', $this->site_id ); - restore_current_blog(); - } + switch_to_blog( $this->site_id ); + $users_of_blog = count_users( 'time', $this->site_id ); + restore_current_blog(); } else { $url = 'users.php'; - if ( $count_users ) { - $users_of_blog = count_users(); - } + $users_of_blog = count_users(); } - 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(); - } + $total_users = $users_of_blog['total_users']; + $avail_roles =& $users_of_blog['avail_roles']; + unset($users_of_blog); $current_link_attributes = empty( $role ) ? ' class="current" aria-current="page"' : ''; $role_links = array(); - - if ( $count_users ) { - $role_links['all'] = "" . sprintf( _nx( 'All (%s)', 'All (%s)', $total_users, 'users' ), number_format_i18n( $total_users ) ) . ''; - } else { - $role_links['all'] = "" . _x( 'All', 'users' ) . ''; - } + $role_links['all'] = "" . sprintf( _nx( 'All (%s)', 'All (%s)', $total_users, 'users' ), number_format_i18n( $total_users ) ) . ''; foreach ( $wp_roles->get_names() as $this_role => $name ) { - if ( $count_users && !isset($avail_roles[$this_role]) ) { + if ( !isset($avail_roles[$this_role]) ) continue; - } $current_link_attributes = ''; @@ -217,14 +196,12 @@ class WP_Users_List_Table extends WP_List_Table { } $name = translate_user_role( $name ); - if ( $count_users ) { - /* translators: User role name with count */ - $name = sprintf( __('%1$s (%2$s)'), $name, number_format_i18n( $avail_roles[$this_role] ) ); - } + /* translators: User role name with count */ + $name = sprintf( __('%1$s (%2$s)'), $name, number_format_i18n( $avail_roles[$this_role] ) ); $role_links[$this_role] = "$name"; } - if ( ! $count_users || ! empty( $avail_roles['none' ] ) ) { + if ( ! empty( $avail_roles['none' ] ) ) { $current_link_attributes = ''; @@ -233,10 +210,8 @@ class WP_Users_List_Table extends WP_List_Table { } $name = __( 'No role' ); - if ( $count_users ) { - /* translators: User role name with count */ - $name = sprintf( __('%1$s (%2$s)'), $name, number_format_i18n( $avail_roles['none' ] ) ); - } + /* translators: User role name with count */ + $name = sprintf( __('%1$s (%2$s)'), $name, number_format_i18n( $avail_roles['none' ] ) ); $role_links['none'] = "$name"; } diff --git a/src/wp-includes/default-filters.php b/src/wp-includes/default-filters.php index a725221381..0ed4be9056 100644 --- a/src/wp-includes/default-filters.php +++ b/src/wp-includes/default-filters.php @@ -466,11 +466,6 @@ add_filter( 'nav_menu_item_id', '_nav_menu_item_id_use_once', 10, 2 ); // Widgets add_action( 'init', 'wp_widgets_init', 1 ); -// User counts -foreach ( array( 'user_register', 'deleted_user' ) as $action ){ - add_action( $action, 'wp_update_active_user_count' ); -} - // Admin Bar // Don't remove. Wrong way to disable. add_action( 'template_redirect', '_wp_admin_bar_init', 0 ); diff --git a/src/wp-includes/functions.php b/src/wp-includes/functions.php index c00e34da9d..2720f590af 100644 --- a/src/wp-includes/functions.php +++ b/src/wp-includes/functions.php @@ -5837,64 +5837,3 @@ All at ###SITENAME### $site_name ), $email_change_email['message'], $email_change_email['headers'] ); } - -/** - * Whether or not we have a large site, based on its number of users. - * - * The default criteria for a large site is more than 10,000 users. - * - * @since 4.9.0 - * - * @return bool True if the site meets the criteria for large. False otherwise. - */ -function wp_is_large_user_count() { - $count = wp_get_active_user_count(); - - /** - * Filters whether the site is considered large, based on its number of users. - * - * The default criteria for a large site is more than 10,000 users. - * - * @since 4.9.0 - * - * @param bool $is_large_user_count Whether the site is considered large. - * @param int $count The number of users on the site. - */ - return apply_filters( 'wp_is_large_user_count', $count > 10000, $count ); -} - -/** - * Update the active user count. - * - * @since 4.9.0 - * @global wpdb $wpdb WordPress database abstraction object. - * - * @return int The active user count. - */ -function wp_update_active_user_count() { - global $wpdb; - - $count = $wpdb->get_var( " - SELECT COUNT(ID) as c - FROM {$wpdb->users} - " ); - update_option( 'active_user_count', $count ); - - return (int) $count; -} - -/** - * The number of active users. - * - * @since 4.9.0 - * - * @return int The active user count. - */ -function wp_get_active_user_count() { - $count = get_option( 'active_user_count', false ); - - if ( false === $count ) { - $count = wp_update_active_user_count(); - } - return (int) $count; -} diff --git a/src/wp-includes/ms-functions.php b/src/wp-includes/ms-functions.php index d4bb42f616..4a6f0ed0ae 100644 --- a/src/wp-includes/ms-functions.php +++ b/src/wp-includes/ms-functions.php @@ -2529,11 +2529,6 @@ function wp_is_large_network( $using = 'sites', $network_id = null ) { if ( 'users' == $using ) { $count = get_user_count( $network_id ); - $is_large = ( $count > 10000 ); - - /** This filter is documented in wp-includes/functions.php */ - $is_large = apply_filters( 'wp_is_large_user_count', $is_large, $count ); - /** * Filters whether the network is considered large. * @@ -2545,7 +2540,7 @@ function wp_is_large_network( $using = 'sites', $network_id = null ) { * @param int $count The count of items for the component. * @param int $network_id The ID of the network being checked. */ - return apply_filters( 'wp_is_large_network', $is_large, 'users', $count, $network_id ); + return apply_filters( 'wp_is_large_network', $count > 10000, 'users', $count, $network_id ); } $count = get_blog_count( $network_id ); diff --git a/src/wp-includes/update.php b/src/wp-includes/update.php index 13c20b7ad8..f178470a36 100644 --- a/src/wp-includes/update.php +++ b/src/wp-includes/update.php @@ -78,7 +78,8 @@ function wp_version_check( $extra_stats = array(), $force_check = false ) { $wp_install = network_site_url(); $multisite_enabled = 1; } else { - $user_count = wp_get_active_user_count(); + $user_count = count_users(); + $user_count = $user_count['total_users']; $multisite_enabled = 0; $num_blogs = 1; $wp_install = home_url( '/' );