Changeset 15539 for trunk/wp-admin/includes/user.php

Timestamp:

08/27/2010 12:18:57 AM (14 years ago)

Author:

scribu

Message:

use get_users() in get_editable_user_ids() and cache result. See #14572

File:

• trunk/wp-admin/includes/user.php (modified) (2 diffs)

trunk/wp-admin/includes/user.php

@@ -241 +241 @@
  *
  * @param int $user_id User ID.
- * @param bool $exclude_zeros Optional, default is true. Whether to exclude zeros.
- * @return unknown
- */
-function get_editable_user_ids( $user_id, $exclude_zeros = true, $post_type = 'post' ) {
-    global $wpdb;
+ * @param bool $deprecated Not used.
+ * @return array
+ */
+function get_editable_user_ids( $user_id, $deprecated = true, $post_type = 'post' ) {
+    global $wpdb;
+
+    if ( !$deprecated )
+        _deprecated_argument( __FUNCTION__, '3.1.0' );
 
     $user = new WP_User( $user_id );
@@ -257 +260 @@
     }
 
-    if ( !is_multisite() )
-        $level_key = $wpdb->get_blog_prefix() . 'user_level';
-    else
-        $level_key = $wpdb->get_blog_prefix() . 'capabilities'; // wpmu site admins don't have user_levels
-
-    $query = $wpdb->prepare("SELECT user_id FROM $wpdb->usermeta WHERE meta_key = %s", $level_key);
-    if ( $exclude_zeros )
-        $query .= " AND meta_value != '0'";
-
-    return $wpdb->get_col( $query );
+    return get_users( array('fields' => 'ids') );
 }