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

WordPress Trac noreply at wordpress.org
Tue Sep 6 00:36:18 UTC 2016


#22744: Media search doesn't include file name
-------------------------------------------------+-------------------------
 Reporter:  johnbillion                          |       Owner:
     Type:  enhancement                          |  wonderboymusic
 Priority:  normal                               |      Status:  reviewing
Component:  Media                                |   Milestone:  4.7
 Severity:  normal                               |     Version:  3.0
 Keywords:  has-patch needs-testing has-unit-    |  Resolution:
  tests                                          |     Focuses:
                                                 |  administration
-------------------------------------------------+-------------------------

Comment (by joemcgill):

 [attachment:22744.8.diff] returns to the `GROUP BY` clause approach from
 [attachment:22744.6.diff] and splits the assertions in the first test into
 two separate unit test.

 @swissspidy I agree that the performance cost, however small, is a knock
 against adding this to the front end. I even wonder if we should avoid
 adding this to the back end as well and let plugins handle what I imagine
 is a fairly limited use case, if we're going to apply the
 [https://wordpress.org/about/philosophy/#majority Design for the Majority]
 principle here.

 @Kent Brockman if we do modify Core's behavior here, we would most likely
 ''not'' add a new setting in the admin to modify this behavior. It would
 have to be overridden programmatically with a filter, as current plugins
 are doing to add this functionality now.

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


More information about the wp-trac mailing list