[wp-trac] [WordPress Trac] #36949: Term exists should use get_terms internally

WordPress Trac noreply at wordpress.org
Tue Jan 8 00:09:15 UTC 2019


#36949: Term exists should use get_terms internally
--------------------------------------+-----------------------------
 Reporter:  spacedmonkey              |       Owner:  (none)
     Type:  enhancement               |      Status:  new
 Priority:  normal                    |   Milestone:  Future Release
Component:  Taxonomy                  |     Version:  2.3
 Severity:  normal                    |  Resolution:
 Keywords:  has-patch has-unit-tests  |     Focuses:  performance
--------------------------------------+-----------------------------
Changes (by spacedmonkey):

 * keywords:  has-patch => has-patch has-unit-tests


Comment:

 I have updated the patch with
 [https://core.trac.wordpress.org/attachment/ticket/36949/36949.diff
 36949.diff].

 Now all tests pass with a very simple work around for cached queries on
 import. Simple disable the caching on import. This means all tests pass.
 The test coverage is pretty good for term_exists, so I believe we may not
 need to add anymore tests.

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


More information about the wp-trac mailing list