Make WP_User_Query::prepare_query() public

[scribu]

WP_User_Query tries to follow the pattern from WP_Query, where if you don't pass any args, it won't run the query.

The problem is that you can't do anything with that instance afterwards; both prepare_query() and query() accept no parameters.

[scribu]

With the patch, it's now possible to do this:

// construct $args somehow

$uq = new WP_User_Query;

$uq->prepare_query( $args );

var_dump( $uq->query_where );

[nacin]

Looks good.

[wonderboymusic]

21119.2.diff​ updates the patch, which was blowing up against trunk. I also made changes so that it passes all unit tests. This class is awkward, may need more unit tests.

[wonderboymusic]

21119.3.diff​ adds unit tests for WP_User_Query::prepare_query() and demonstrates how it works. It only does something if WP_User_Query::query_vars is empty or an array of any options is passed. Otherwise, it rebuilds the object properties from the previously set query vars.

[wonderboymusic]

In 27185:

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.

[nacin]

#27741 was marked as a duplicate.

[nacin]

As noted in #27741, these unit tests fail under multisite. Multisite does indeed have a quirk here as it relates to WP_User_Query::prepare_query(), and I don't think it's the test that's the problem, but rather, this new public method.

The final $query->prepare_query() call re-does all of the logic in prepare_query(), re-adding another meta query for is_multisite(). This makes this function not safe to be called a second time, and I believe the intention was that it should.

[meta_query] => Array(
	[0] => Array( [key] => wptests_capabilities )
	[1] => Array( [key] => wptests_capabilities )
)

On the bright side, this doesn't *appear* to break anything. We should just be able to just look for an existing duplicate meta query and discard adding a new one. Patch attached, 21119.4.diff​. As it doesn't seem to break anything as-is, I could spin this into a new ticket for post-3.9 if necessary.

[scribu]

+1 for 21119.4.diff​. It was an oversight in the initial patch.

[nacin]

Reviewed by markjaquith. Good to go.

[nacin]

In 28087:

User Query: Don't blindly re-append new meta queries for capabilities.

fixes #21119.