Document WP_Query and related class arguments

[DrewAPicture]

It's been a long time coming, but here we are.

We've never had a fully documented list of WP_Query and related class arguments in core docs, however, with the inline docs effort in full swing, and the Code Reference on the way, there's an increasing urgency to have this list in place.

And there's no better time to start than now.

Goals of this ticket:

1. Document all accepted WP_Query arguments
2. Document all related class arguments, including:

• WP_Tax_Query arguments
• WP_Date_Query arguments
• WP_Meta_Query arguments

Goals of a future ticket:

1. Remove duplicated WP_Query argument docs elsewhere in core inline docs.
2. Back-reference mentions of accepting WP_Query parameters to the WP_Query docs

The attached patch is a relatively complete start to this effort.

Constructive input is welcome and encouraged for hashing out:

1. What arguments should or shouldn't be documented
2. Descriptions of said arguments in WP_Query or related classes
3. Defaults and Accepted values for said arguments

Let's get to it.

[DrewAPicture]

First-run for WP_Query, WP_Tax_Query, WP_Date_Query, WP_Meta_Query

[rmccue]

+1, this is awesome and will save me a bunch of time documenting it for the REST API. <3

[Viper007Bond]

Awesome. Nice work!

Here's my feedback on the WP_Date_Query stuff:

• If we're going to document the class via it's constructor, then it probably makes sense to remove a large portion of the PHPdoc that's at the top of the class. No need to duplicate.
• Probably worth mentioning that the list of acceptable columns is filterable.
• For before and after, if you're passing an array, then the month and day fields are optional (you don't have to supply all three). For before it defaults to January and/or the first day of the month and for after, it defaults to December and/or the last day of the month. This way you can just say "before 2013" or "before June 2013".
• I think we can improve the wording for inclusive, but I'm not sure exactly how.
• The "accepts" for $default_column is missing.

[DrewAPicture]

I think we're going to need a lot more eyes-on the WP_Query stuff before I'm happy enough to put it in. That said, I'd like to get the WP_Date_Query inline docs in for 3.7, then push the rest down the road.

[johnbillion]

Can we split this ticket into WP_Query and WP_Date_Query?

[DrewAPicture]

Moving to 3.8. Let's split out the WP_Date_Query docs as suggested by @johnbillion. I'll make a new ticket.

[DrewAPicture]

Opened #25552 for WP_Date_Query documentation.

[DrewAPicture]

Sad punt face.

[GaryJ]

Patch needs updating to remove WP_Date_Query docs which have been committed via the other ticket.

Worth creating separate tickets for Taxonomy and Meta query docs, and getting them in, rather than trying to do it all under this ticket?

[DrewAPicture]

Replying to GaryJ:

Patch needs updating to remove WP_Date_Query docs which have been committed via the other ticket.

Worth creating separate tickets for Taxonomy and Meta query docs, and getting them in, rather than trying to do it all under this ticket?

That's fine, though I think I'd rather we/you/they wait to create each ticket until there's a patch to go with it.

[DrewAPicture]

WP_Query only

[DrewAPicture]

25367.2.diff​ refreshes for WP_Query arguments only.

[DrewAPicture]

[28887] is a first run of the WP_Query args. Looks like it missed trac.

[DrewAPicture]

I think we're good here.

[SergeyBiryukov]

In 29538:

Correct 'no_found_rows' parameter description.

see #25367.