diff options
author | donncha <donncha@7be80a69-a1ef-0310-a953-fb0f7c49ff36> | 2008-05-15 16:35:59 +0000 |
---|---|---|
committer | donncha <donncha@7be80a69-a1ef-0310-a953-fb0f7c49ff36> | 2008-05-15 16:35:59 +0000 |
commit | cb6323b69dbd6e62362f12fb7f474f1707bf3f09 (patch) | |
tree | 6574ccf2276c1d8faca8e1247720ffadb881bbe8 /wp-admin/wpmu-users.php | |
parent | 8fdae25acec9304cb13a9f0cb82316e1f6a9f7c9 (diff) | |
download | wordpress-mu-cb6323b69dbd6e62362f12fb7f474f1707bf3f09.tar.gz wordpress-mu-cb6323b69dbd6e62362f12fb7f474f1707bf3f09.tar.xz wordpress-mu-cb6323b69dbd6e62362f12fb7f474f1707bf3f09.zip |
Whitespace cleanup.
Clean up parameter filtering
git-svn-id: http://svn.automattic.com/wordpress-mu/trunk@1289 7be80a69-a1ef-0310-a953-fb0f7c49ff36
Diffstat (limited to 'wp-admin/wpmu-users.php')
-rw-r--r-- | wp-admin/wpmu-users.php | 39 |
1 files changed, 20 insertions, 19 deletions
diff --git a/wp-admin/wpmu-users.php b/wp-admin/wpmu-users.php index 1305e09..47238b9 100644 --- a/wp-admin/wpmu-users.php +++ b/wp-admin/wpmu-users.php @@ -40,18 +40,19 @@ if ( $_GET['updated'] == 'true' ) { <?php $apage = isset( $_GET['apage'] ) ? intval( $_GET['apage'] ) : 1; $num = isset( $_GET['num'] ) ? intval( $_GET['num'] ) : 15; + $s = wp_specialchars( trim( $_GET[ 's' ] ) ); $query = "SELECT * FROM {$wpdb->users}"; - - if( !empty($_GET['s']) ) { - $search = '%' . trim(addslashes($_GET['s'])) . '%'; + + if( !empty( $s ) ) { + $search = '%' . trim( $s ) . '%'; $query .= " WHERE user_login LIKE '$search' OR user_email LIKE '$search'"; } - + if( !isset($_GET['sortby']) ) { $_GET['sortby'] = 'id'; } - + if( $_GET['sortby'] == 'email' ) { $query .= ' ORDER BY user_email '; } elseif( $_GET['sortby'] == 'id' ) { @@ -63,18 +64,18 @@ if ( $_GET['updated'] == 'true' ) { } elseif( $_GET['sortby'] == 'registered' ) { $query .= ' ORDER BY user_registered '; } - + $query .= ( $_GET['order'] == 'DESC' ) ? 'DESC' : 'ASC'; - if( !empty($_GET['s'])) { + if( !empty( $s )) { $user_list = $wpdb->get_results( $query, ARRAY_A ); - $total = count($user_list); + $total = count($user_list); } else { $total = $wpdb->get_var( "SELECT COUNT(*) FROM {$wpdb->users}"); } - + $query .= " LIMIT " . intval( ( $apage - 1 ) * $num) . ", " . intval( $num ); - + $user_list = $wpdb->get_results( $query, ARRAY_A ); // Pagination @@ -87,13 +88,13 @@ if ( $_GET['updated'] == 'true' ) { ?> <h2><?php _e("Users"); ?></h2> <form action="wpmu-users.php" method="get" style="position:absolute;right:0;top:0;"> - <input type="text" name="s" value="<?php if (isset($_GET['s'])) echo stripslashes(wp_specialchars($_GET['s'], 1)); ?>" size="17" /> + <input type="text" name="s" value="<?php if (isset($_GET['s'])) echo stripslashes( $s ); ?>" size="17" /> <input type="submit" id="post-query-submit" value="<?php _e('Search Users') ?>" class="button" /> </form> <form id="form-user-list" action='wpmu-edit.php?action=allusers' method='post'> <div class="tablenav"> - <?php if ( $user_navigation ) echo "<div class='tablenav-pages'>$user_navigation</div>"; ?> + <?php if ( $user_navigation ) echo "<div class='tablenav-pages'>$user_navigation</div>"; ?> <div class="alignleft"> <input type="submit" value="<?php _e('Delete') ?>" name="alluser_delete" class="button-secondary delete" /> @@ -105,9 +106,9 @@ if ( $_GET['updated'] == 'true' ) { </div> <br class="clear" /> - + <?php if( isset($_GET['s']) && $_GET['s'] != '' ) : ?> - <p><a href="wpmu-blogs.php?action=blogs&s=<?php echo stripslashes(wp_specialchars($_GET['s'], 1)); ?>"><?php _e('Search Blogs:') ?> <strong><?php echo stripslashes(wp_specialchars($_GET['s'], 1)) ?></strong></a></p> + <p><a href="wpmu-blogs.php?action=blogs&s=<?php echo urlencode( stripslashes( $s ) ); ?>"><?php _e('Search Blogs:') ?> <strong><?php echo stripslashes( $s ) ?></strong></a></p> <?php endif; ?> <?php @@ -143,7 +144,7 @@ if ( $_GET['updated'] == 'true' ) { foreach ( (array) $user_list as $user) { $class = ('alternate' == $class) ? '' : 'alternate'; ?> - + <tr class="<?php echo $class; ?>"> <?php foreach( (array) $posts_columns as $column_name=>$column_display_name) : @@ -152,8 +153,8 @@ if ( $_GET['updated'] == 'true' ) { <th scope="row" class="check-column"><input type='checkbox' id='user_<?php echo $user['ID'] ?>' name='allusers[]' value='<?php echo $user['ID'] ?>' /></th> <?php break; - - case 'id': ?> + + case 'id': ?> <td><?php echo $user['ID'] ?></td> <?php break; @@ -228,11 +229,11 @@ if( apply_filters('show_adduser_fields', true) ) : <h2><?php _e('Add user') ?></h2> <form action="wpmu-edit.php?action=adduser" method="post"> <table class="form-table"> - <tr class="form-field form-required"> + <tr class="form-field form-required"> <th scope='row'><?php _e('Username') ?></th> <td><input type="text" name="user[username]" /></td> </tr> - <tr class="form-field form-required"> + <tr class="form-field form-required"> <th scope='row'><?php _e('Email') ?></th> <td><input type="text" name="user[email]" /></td> </tr> |