Changeset 27185 for trunk/src/wp-includes/user.php

Timestamp:

02/17/2014 09:40:04 PM (12 years ago)

Author:

wonderboymusic

Message:

Make WP_User_Query::prepare_query() public by allowing it to be passed an array of args. Previously, if the WP_User_Query constructor was not passed args, the object was basically unusable. Adds unit tests, all other tests pass.

Props scribu, for the initial patch.
Fixes #21119.

File:

• trunk/src/wp-includes/user.php (modified) (4 diffs)

trunk/src/wp-includes/user.php

@@ -432 +432 @@
      */
     function __construct( $query = null ) {
-        if ( !empty( $query ) ) {
+        if ( ! empty( $query ) ) {
+            $this->prepare_query( $query );
+            $this->query();
+        }
+    }
+
+    /**
+     * Prepare the query variables
+     *
+     * @since 3.1.0
+     *
+     * @param string|array $args The query variables
+     */
+    function prepare_query( $query = array() ) {
+        global $wpdb;
+
+        if ( empty( $this->query_vars ) || ! empty( $query ) ) {
+            $this->query_limit = null;
             $this->query_vars = wp_parse_args( $query, array(
                 'blog_id' => $GLOBALS['blog_id'],
@@ -451 +468 @@
                 'who' => ''
             ) );
-
-            $this->prepare_query();
-            $this->query();
-        }
-    }
-
-    /**
-     * Prepare the query variables
-     *
-     * @since 3.1.0
-     * @access private
-     */
-    function prepare_query() {
-        global $wpdb;
+        }
 
         $qv =& $this->query_vars;
@@ -650 +654 @@
      *
      * @since 3.1.0
-     * @access private
      */
     function query() {
@@ -657 +660 @@
         $qv =& $this->query_vars;
 
+        $query = "SELECT $this->query_fields $this->query_from $this->query_where $this->query_orderby $this->query_limit";
+
         if ( is_array( $qv['fields'] ) || 'all' == $qv['fields'] ) {
-            $this->results = $wpdb->get_results("SELECT $this->query_fields $this->query_from $this->query_where $this->query_orderby $this->query_limit");
+            $this->results = $wpdb->get_results( $query );
         } else {
-            $this->results = $wpdb->get_col("SELECT $this->query_fields $this->query_from $this->query_where $this->query_orderby $this->query_limit");
+            $this->results = $wpdb->get_col( $query );
         }