From 198dee542ddd1c388fee1e5d80d01b65bcc4e72e Mon Sep 17 00:00:00 2001
From: Boone Gorges <boonebgorges@gmail.com>
Date: Sat, 3 Oct 2015 18:45:25 +0000
Subject: [PATCH] Ensure that `WP_User_Query` vars are filled after
 'pre_get_users'.

This prevents notices from being thrown when a 'pre_get_users' callback
removes required values from the list of `query_vars`.

For backward compatibility with previous uses of 'pre_get_users', default
values are parsed both before and after the action is fired.

Fixes #33449.
Built from https://develop.svn.wordpress.org/trunk@34804


git-svn-id: http://core.svn.wordpress.org/trunk@34769 1a063a9b-81f0-0310-95a4-ce76da25c4cd
---
 wp-includes/class-wp-user-query.php | 57 +++++++++++++++++++----------
 wp-includes/version.php             |  2 +-
 2 files changed, 38 insertions(+), 21 deletions(-)

diff --git a/wp-includes/class-wp-user-query.php b/wp-includes/class-wp-user-query.php
index 21d013ca28..1bc08aa491 100644
--- a/wp-includes/class-wp-user-query.php
+++ b/wp-includes/class-wp-user-query.php
@@ -75,6 +75,40 @@ class WP_User_Query {
 		}
 	}
 
+	/**
+	 * Fills in missing query variables with default values.
+	 *
+	 * @since 4.4.0
+	 * @access public
+	 *
+	 * @param array $args Query vars, as passed to `WP_User_Query`.
+	 * @return array Complete query variables with undefined ones filled in with defaults.
+	 */
+	public static function fill_query_vars( $args ) {
+		$defaults = array(
+			'blog_id' => $GLOBALS['blog_id'],
+			'role' => '',
+			'meta_key' => '',
+			'meta_value' => '',
+			'meta_compare' => '',
+			'include' => array(),
+			'exclude' => array(),
+			'search' => '',
+			'search_columns' => array(),
+			'orderby' => 'login',
+			'order' => 'ASC',
+			'offset' => '',
+			'number' => '',
+			'paged' => 1,
+			'count_total' => true,
+			'fields' => 'all',
+			'who' => '',
+			'has_published_posts' => null,
+		);
+
+		return wp_parse_args( $args, $defaults );
+	}
+
 	/**
 	 * Prepare the query variables.
 	 *
@@ -146,26 +180,7 @@ class WP_User_Query {
 
 		if ( empty( $this->query_vars ) || ! empty( $query ) ) {
 			$this->query_limit = null;
-			$this->query_vars = wp_parse_args( $query, array(
-				'blog_id' => $GLOBALS['blog_id'],
-				'role' => '',
-				'meta_key' => '',
-				'meta_value' => '',
-				'meta_compare' => '',
-				'include' => array(),
-				'exclude' => array(),
-				'search' => '',
-				'search_columns' => array(),
-				'orderby' => 'login',
-				'order' => 'ASC',
-				'offset' => '',
-				'number' => '',
-				'paged' => 1,
-				'count_total' => true,
-				'fields' => 'all',
-				'who' => '',
-				'has_published_posts' => null,
-			) );
+			$this->query_vars = $this->fill_query_vars( $query );
 		}
 
 		/**
@@ -181,7 +196,9 @@ class WP_User_Query {
 		 */
 		do_action( 'pre_get_users', $this );
 
+		// Ensure that query vars are filled after 'pre_get_users'.
 		$qv =& $this->query_vars;
+		$qv =  $this->fill_query_vars( $qv );
 
 		if ( is_array( $qv['fields'] ) ) {
 			$qv['fields'] = array_unique( $qv['fields'] );
diff --git a/wp-includes/version.php b/wp-includes/version.php
index 2e82873e8a..ce6532f038 100644
--- a/wp-includes/version.php
+++ b/wp-includes/version.php
@@ -4,7 +4,7 @@
  *
  * @global string $wp_version
  */
-$wp_version = '4.4-alpha-34803';
+$wp_version = '4.4-alpha-34804';
 
 /**
  * Holds the WordPress DB revision, increments when changes are made to the WordPress DB schema.