[wp-trac] [WordPress Trac] #35791: WP_Site_Query class
WordPress Trac
noreply at wordpress.org
Wed May 11 19:40:04 UTC 2016
#35791: WP_Site_Query class
-------------------------------------------------+-------------------------
Reporter: spacedmonkey | Owner: jeremyfelt
Type: task (blessed) | Status: reviewing
Priority: normal | Milestone: 4.6
Component: Networks and Sites | Version: 4.4
Severity: normal | Resolution:
Keywords: has-patch needs-testing has-unit- | Focuses: multisite
tests |
-------------------------------------------------+-------------------------
Comment (by jeremyfelt):
In [attachment:35791-class.5.patch]:
* Updated the naming of the individual tests to
`test_wp_site_query_by_{argument}_with_{descriptive}`.
* Added a series of tests to continue building out coverage for more query
args. Date queries should be the only thing left, though it wouldn't hurt
to think of more scenarios to test.
* I've removed the `__call()` method as we don't have the same back-compat
requirements for the protected `get_search_sql` method.
* This may be a future improvement, but it would be cool if we could pass
`wordpress.org/foo/` to `search` and have it split the domain and path
into 2 search parts.
--
Ticket URL: <https://core.trac.wordpress.org/ticket/35791#comment:41>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform
More information about the wp-trac
mailing list