Changeset 20631

Timestamp:

04/27/2012 11:13:39 PM (14 years ago)

Author:

nacin

Message:

After adding a user, take them back to users.php with an 'Edit user' link in the message.

This removes the (possibly ineffective) search from the redirect, as well as the URL hash that will get covered up by the toolbar.

fixes #19470. props merty and SergeyBiryukov for initial patches.

Location:

trunk/wp-admin

Files:

• user-new.php (modified) (1 diff)
• users.php (modified) (1 diff)

trunk/wp-admin/user-new.php

@@ -99 +99 @@
                         $add_user_errors = $user_id;
                 } else {
-                        if ( current_user_can('edit_users') ) {
-                                $new_user_login = apply_filters('pre_user_login', sanitize_user(stripslashes($_REQUEST['user_login']), true));
-                                $redirect = 'users.php?usersearch='. urlencode($new_user_login) . '&update=add' . '#user-' . $user_id;
-                        } else {
+                        if ( current_user_can( 'list_users' ) )
+                                $redirect = 'users.php?update=add&id=' . $user_id;
+                        else
                                 $redirect = add_query_arg( 'update', 'add', 'user-new.php' );
-                        }
                         wp_redirect( $redirect );
                         die();

trunk/wp-admin/users.php

@@ -354 +354 @@
                 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':