[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