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

Timestamp:

02/17/2014 09:40:04 PM (13 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 );
                 }