Changeset 7057 for trunk/wp-admin/users.php

Timestamp:

02/27/2008 12:46:27 AM (18 years ago)

Author:

ryan

Message:

Don't load wp-lists on pages that don't need it. Props mdawaffe. fixes #6011

File:

• trunk/wp-admin/users.php (modified) (7 diffs)

trunk/wp-admin/users.php

@@ -265 +265 @@
 unset($users_of_blog);
 
+$current_role = false;
 foreach ( $wp_roles->get_names() as $role => $name ) {
     if ( !isset($avail_roles[$role]) )
@@ -271 +272 @@
     $class = '';
 
-    if ( $role == $_GET['role'] )
+    if ( $role == $_GET['role'] ) {
+        $current_role = $_GET['role'];
         $class = ' class="current"';
+    }
 
     $name = translate_with_context($name);
@@ -340 +343 @@
 <tbody id="users" class="list:user user-list">
 <?php
+$style = '';
 foreach ( $wp_user_search->get_results() as $userid ) {
     $user_object = new WP_User($userid);
@@ -345 +349 @@
     $role = array_shift($roles);
 
-    $style = '';
     $style = ( ' class="alternate"' == $style ) ? '' : ' class="alternate"';
     echo "\n\t" . user_row($user_object, $style, $role);
@@ -400 +403 @@
             echo '<p>' . sprintf(__('Users cannot currently <a href="%1$s">register themselves</a>, but you can manually create users here.'), get_option('siteurl').'/wp-admin/options-general.php#users_can_register') . '</p>';
 ?>
-<form action="#add-new-user" method="post" name="adduser" id="adduser" class="add:user-list:">
+<form action="#add-new-user" method="post" name="adduser" id="adduser" class="add:users:">
 <?php wp_nonce_field('add-user') ?>
 <table class="form-table">
@@ -438 +441 @@
             <?php
             if ( !$new_user_role )
-                $new_user_role = get_option('default_role');
+                $new_user_role = $current_role ? $current_role : get_option('default_role');
             wp_dropdown_roles($new_user_role);
             ?>
@@ -451 +454 @@
 </form>
 
-<table style="color:red">
-<tbody id="user-list" class="list:user">
-</tbody>
-</table>
-
 </div>