Users: Revert [41613], [41614], and [41623] as this feature needs some more work.

See #38741

Built from https://develop.svn.wordpress.org/trunk@41753


git-svn-id: http://core.svn.wordpress.org/trunk@41587 1a063a9b-81f0-0310-95a4-ce76da25c4cd
This commit is contained in:
John Blackbourn 2017-10-04 22:17:46 +00:00
parent adb8f0161b
commit 50948669eb
6 changed files with 18 additions and 113 deletions

View File

@ -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'] = "<a href='$url'$current_link_attributes>" . 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'$current_link_attributes>" . _x( 'All', 'users' ) . '</a>';
}
$role_links['all'] = "<a href='$url'$current_link_attributes>" . sprintf( _nx( 'All <span class="count">(%s)</span>', 'All <span class="count">(%s)</span>', $total_users, 'users' ), number_format_i18n( $total_users ) ) . '</a>';
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 <span class="count">(%2$s)</span>'), $name, number_format_i18n( $avail_roles[$this_role] ) );
}
/* 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 ) ) . "'$current_link_attributes>$name</a>";
}
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 <span class="count">(%2$s)</span>'), $name, number_format_i18n( $avail_roles['none' ] ) );
}
/* 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 ) ) . "'$current_link_attributes>$name</a>";
}

View File

@ -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 );

View File

@ -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;
}

View File

@ -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 );

View File

@ -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( '/' );

View File

@ -4,7 +4,7 @@
*
* @global string $wp_version
*/
$wp_version = '4.9-alpha-41752';
$wp_version = '4.9-alpha-41753';
/**
* Holds the WordPress DB revision, increments when changes are made to the WordPress DB schema.