WordPress 3.5.1
[autoinstalls/wordpress.git] / wp-admin / users.php
index 5fefb4276ef3526e90b46d79a908307c27daf3c9..6ea17657c15d4f3fd748cceafb7238c8211c3b5f 100644 (file)
@@ -33,7 +33,7 @@ get_current_screen()->add_help_tab( array(
        'content' => '<p>' . __('You can customize the display of this screen in a number of ways:') . '</p>' .
                                        '<ul>' .
                                        '<li>' . __('You can hide/display columns based on your needs and decide how many users to list per screen using the Screen Options tab.') . '</li>' .
-                                       '<li>' . __('You can filter the list of users by User Role using the text links in the upper left to show All, Administrator, Editor, Author, Contributor, or Subscriber. The default view is to show all users.  Unused User Roles are not listed.') . '</li>' .
+                                       '<li>' . __('You can filter the list of users by User Role using the text links in the upper left to show All, Administrator, Editor, Author, Contributor, or Subscriber. The default view is to show all users. Unused User Roles are not listed.') . '</li>' .
                                        '<li>' . __('You can view all posts made by a user by clicking on the number under the Posts column.') . '</li>' .
                                        '</ul>'
 ) );
@@ -75,6 +75,22 @@ if ( empty($_REQUEST) ) {
 
 $update = '';
 
+/**
+ * @since 3.5.0
+ * @access private
+ */
+function delete_users_add_js() { ?>
+<script>
+jQuery(document).ready( function($) {
+       var submit = $('#submit').prop('disabled', true);
+       $('input[name=delete_option]').one('change', function() {
+               submit.prop('disabled', false);
+       });
+});
+</script>
+<?php
+}
+
 switch ( $wp_list_table->current_action() ) {
 
 /* Bulk Dropdown menu Role changes */
@@ -111,7 +127,7 @@ case 'promote':
                if ( is_multisite() && !is_user_member_of_blog( $id ) )
                        wp_die(__('Cheatin&#8217; uh?'));
 
-               $user = new WP_User($id);
+               $user = get_userdata( $id );
                $user->set_role($_REQUEST['new_role']);
        }
 
@@ -131,16 +147,22 @@ case 'dodelete':
                exit();
        }
 
+       $userids = array_map( 'intval', (array) $_REQUEST['users'] );
+
+       if ( empty( $_REQUEST['delete_option'] ) ) {
+               $url = self_admin_url( 'users.php?action=delete&users[]=' . implode( '&users[]=', $userids ) . '&error=true' );
+               $url = str_replace( '&amp;', '&', wp_nonce_url( $url, 'bulk-users' ) );
+               wp_redirect( $url );
+               exit;
+       }
+
        if ( ! current_user_can( 'delete_users' ) )
                wp_die(__('You can&#8217;t delete users.'));
 
-       $userids = $_REQUEST['users'];
        $update = 'del';
        $delete_count = 0;
 
-       foreach ( (array) $userids as $id) {
-               $id = (int) $id;
-
+       foreach ( $userids as $id ) {
                if ( ! current_user_can( 'delete_user', $id ) )
                        wp_die(__( 'You can&#8217;t delete that user.' ) );
 
@@ -150,12 +172,10 @@ case 'dodelete':
                }
                switch ( $_REQUEST['delete_option'] ) {
                case 'delete':
-                       if ( current_user_can('delete_user', $id) )
-                               wp_delete_user($id);
+                       wp_delete_user( $id );
                        break;
                case 'reassign':
-                       if ( current_user_can('delete_user', $id) )
-                               wp_delete_user($id, $_REQUEST['reassign_user']);
+                       wp_delete_user( $id, $_REQUEST['reassign_user'] );
                        break;
                }
                ++$delete_count;
@@ -182,9 +202,11 @@ case 'delete':
                $errors = new WP_Error( 'edit_users', __( 'You can&#8217;t delete users.' ) );
 
        if ( empty($_REQUEST['users']) )
-               $userids = array(intval($_REQUEST['user']));
+               $userids = array( intval( $_REQUEST['user'] ) );
        else
-               $userids = (array) $_REQUEST['users'];
+               $userids = array_map( 'intval', (array) $_REQUEST['users'] );
+
+       add_action( 'admin_head', 'delete_users_add_js' );
 
        include ('admin-header.php');
 ?>
@@ -195,29 +217,33 @@ case 'delete':
 <div class="wrap">
 <?php screen_icon(); ?>
 <h2><?php _e('Delete Users'); ?></h2>
+<?php if ( isset( $_REQUEST['error'] ) ) : ?>
+<div class="error">
+       <p><strong><?php _e( 'ERROR:' ); ?></strong> <?php _e( 'Please select an option.' ); ?></p>
+</div>
+<?php endif; ?>
 <p><?php echo _n( 'You have specified this user for deletion:', 'You have specified these users for deletion:', count( $userids ) ); ?></p>
 <ul>
 <?php
        $go_delete = 0;
        foreach ( $userids as $id ) {
-               $id = (int) $id;
-               $user = new WP_User($id);
+               $user = get_userdata( $id );
                if ( $id == $current_user->ID ) {
-                       echo "<li>" . sprintf(__('ID #%1s: %2s <strong>The current user will not be deleted.</strong>'), $id, $user->user_login) . "</li>\n";
+                       echo "<li>" . sprintf(__('ID #%1$s: %2$s <strong>The current user will not be deleted.</strong>'), $id, $user->user_login) . "</li>\n";
                } else {
-                       echo "<li><input type=\"hidden\" name=\"users[]\" value=\"" . esc_attr($id) . "\" />" . sprintf(__('ID #%1s: %2s'), $id, $user->user_login) . "</li>\n";
+                       echo "<li><input type=\"hidden\" name=\"users[]\" value=\"" . esc_attr($id) . "\" />" . sprintf(__('ID #%1$s: %2$s'), $id, $user->user_login) . "</li>\n";
                        $go_delete++;
                }
        }
        ?>
        </ul>
 <?php if ( $go_delete ) : ?>
-       <fieldset><p><legend><?php echo _n( 'What should be done with posts and links owned by this user?', 'What should be done with posts and links owned by these users?', $go_delete ); ?></legend></p>
+       <fieldset><p><legend><?php echo _n( 'What should be done with posts owned by this user?', 'What should be done with posts owned by these users?', $go_delete ); ?></legend></p>
        <ul style="list-style:none;">
-               <li><label><input type="radio" id="delete_option0" name="delete_option" value="delete" checked="checked" />
-               <?php _e('Delete all posts and links.'); ?></label></li>
+               <li><label><input type="radio" id="delete_option0" name="delete_option" value="delete" />
+               <?php _e('Delete all posts.'); ?></label></li>
                <li><input type="radio" id="delete_option1" name="delete_option" value="reassign" />
-               <?php echo '<label for="delete_option1">'.__('Attribute all posts and links to:').'</label>';
+               <?php echo '<label for="delete_option1">' . __( 'Attribute all posts to:' ) . '</label> ';
                wp_dropdown_users( array( 'name' => 'reassign_user', 'exclude' => array_diff( $userids, array($current_user->ID) ) ) ); ?></li>
        </ul></fieldset>
        <input type="hidden" name="action" value="dodelete" />
@@ -302,13 +328,13 @@ case 'remove':
        $go_remove = false;
        foreach ( $userids as $id ) {
                $id = (int) $id;
-               $user = new WP_User($id);
+               $user = get_userdata( $id );
                if ( $id == $current_user->ID && !is_super_admin() ) {
-                       echo "<li>" . sprintf(__('ID #%1s: %2s <strong>The current user will not be removed.</strong>'), $id, $user->user_login) . "</li>\n";
+                       echo "<li>" . sprintf(__('ID #%1$s: %2$s <strong>The current user will not be removed.</strong>'), $id, $user->user_login) . "</li>\n";
                } elseif ( !current_user_can('remove_user', $id) ) {
-                       echo "<li>" . sprintf(__('ID #%1s: %2s <strong>You don\'t have permission to remove this user.</strong>'), $id, $user->user_login) . "</li>\n";
+                       echo "<li>" . sprintf(__('ID #%1$s: %2$s <strong>You don\'t have permission to remove this user.</strong>'), $id, $user->user_login) . "</li>\n";
                } else {
-                       echo "<li><input type=\"hidden\" name=\"users[]\" value=\"{$id}\" />" . sprintf(__('ID #%1s: %2s'), $id, $user->user_login) . "</li>\n";
+                       echo "<li><input type=\"hidden\" name=\"users[]\" value=\"{$id}\" />" . sprintf(__('ID #%1$s: %2$s'), $id, $user->user_login) . "</li>\n";
                        $go_remove = true;
                }
        }
@@ -338,6 +364,7 @@ default:
                wp_redirect( add_query_arg( 'paged', $total_pages ) );
                exit;
        }
+
        include('./admin-header.php');
 
        $messages = array();
@@ -346,10 +373,16 @@ default:
                case 'del':
                case 'del_many':
                        $delete_count = isset($_GET['delete_count']) ? (int) $_GET['delete_count'] : 0;
-                       $messages[] = '<div id="message" class="updated"><p>' . sprintf(_n('%s user deleted', '%s users deleted', $delete_count), $delete_count) . '</p></div>';
+                       $messages[] = '<div id="message" class="updated"><p>' . sprintf( _n( 'User deleted.', '%s users deleted.', $delete_count ), number_format_i18n( $delete_count ) ) . '</p></div>';
                        break;
                case 'add':
-                       $messages[] = '<div id="message" class="updated"><p>' . __('New user created.') . '</p></div>';
+                       if ( isset( $_GET['id'] ) && ( $user_id = $_GET['id'] ) && current_user_can( 'edit_user', $user_id ) ) {
+                               $messages[] = '<div id="message" class="updated"><p>' . sprintf( __( 'New user created. <a href="%s">Edit user</a>' ),
+                                       esc_url( add_query_arg( 'wp_http_referer', urlencode( stripslashes( $_SERVER['REQUEST_URI'] ) ),
+                                               self_admin_url( 'user-edit.php?user_id=' . $user_id ) ) ) ) . '</p></div>';
+                       } else {
+                               $messages[] = '<div id="message" class="updated"><p>' . __( 'New user created.' ) . '</p></div>';
+                       }
                        break;
                case 'promote':
                        $messages[] = '<div id="message" class="updated"><p>' . __('Changed roles.') . '</p></div>';