[wp-trac] [WordPress Trac] #38922: Use REST API for ajax tag search

WordPress Trac noreply at wordpress.org
Fri Apr 28 14:54:37 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):

 Reviewing this further I noticed that we dropped two context parameters
 (tax and search) from the `term_search_min_chars` filter...  before
 `apply_filters( 'term_search_min_chars', 2, $tax, $s );` after
 `apply_filters( 'term_search_min_chars', 2 )` - I think this is what
 @nacin was referring to in the ticket description.

 It may be fine to drop these context parameters; if we do we need to
 document that change.  I'm going to try to scan the plugin repository for
 use of this filter and also look at when the filter was introduced if the
 parameters are valuable.

--
Ticket URL: <https://core.trac.wordpress.org/ticket/38922#comment:20>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform


More information about the wp-trac mailing list