[wp-trac] [WordPress Trac] #38922: Use REST API for ajax tag search
WordPress Trac
noreply at wordpress.org
Fri Apr 28 19:57:52 UTC 2017
#38922: Use REST API for ajax tag search
------------------------------+------------------------------
Reporter: nacin | Owner:
Type: defect (bug) | Status: new
Priority: normal | Milestone: Awaiting Review
Component: Taxonomy | Version:
Severity: normal | Resolution:
Keywords: has-patch commit | Focuses: rest-api
------------------------------+------------------------------
Comment (by adamsilverstein):
@DrewAPicture can you recommend the proper way to document removing a
filter's context parameters? Anything beyond adding a description of when
and why the parameters were removed in the docblock? Thinking something
like:
`Previous to 4.8.0 passed taxonomy and search context parameters to the
callback. Use the 'rest_prepare_{$this->taxonomy}' filter to adjust
responses instead.`
What do you think?
any concerns about removing the context parameters given that it is unused
by plugins and developers could now use the
`rest_prepare_{$this->taxonomy}` to similarly filter search min chars
based on search and taxonomy?
--
Ticket URL: <https://core.trac.wordpress.org/ticket/38922#comment:22>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform
More information about the wp-trac
mailing list