[wp-trac] [WordPress Trac] #22744: Media search doesn't include file name

WordPress Trac noreply at wordpress.org
Sun Oct 2 21:59:25 UTC 2016


#22744: Media search doesn't include file name
---------------------------------------------+-----------------------------
 Reporter:  johnbillion                      |       Owner:  joemcgill
     Type:  enhancement                      |      Status:  reopened
 Priority:  normal                           |   Milestone:  4.7
Component:  Media                            |     Version:  3.0
 Severity:  normal                           |  Resolution:
 Keywords:  has-patch has-unit-tests commit  |     Focuses:  administration
---------------------------------------------+-----------------------------
Changes (by joemcgill):

 * keywords:  has-patch has-unit-tests needs-testing => has-patch has-unit-
     tests commit


Comment:

 >  Other than testing these two functions, I don't currently see a way to
 test this in a "larger scope".

 @flixos90: you're correct, I was curious about testing to make sure the
 filters are being applied at the right times, but I can't really think of
 a clean way to do that. In the mean time, I've added another unit test in
 [attachment:22744.13.diff] that ensures things are working when a
 `tax_query` in included, as you suggested. Let me know if you notice
 anything else. Otherwise, I plan to commit this early this week.

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


More information about the wp-trac mailing list