[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