[wp-trac] [WordPress Trac] #30788: Media library views: inconsistent behaviour with mime type groups
WordPress Trac
noreply at wordpress.org
Wed Dec 31 15:26:16 UTC 2014
#30788: Media library views: inconsistent behaviour with mime type groups
--------------------------+------------------------------
Reporter: mdgl | Owner:
Type: defect (bug) | Status: new
Priority: normal | Milestone: Awaiting Review
Component: Media | Version: 4.0
Severity: normal | Resolution:
Keywords: 2nd-opinion | Focuses: javascript
--------------------------+------------------------------
Comment (by mdgl):
I did a lot more testing and code review on this today and am now much
more confident in the (updated) patch.
It turns out that `wp_match_mime_types()` is also used to match what are
effectively file names and types by function `wp_mime_type_icon()` so I
have added back the third (and un-anchored) regular expression to retain
backwards compatibility. The existing code in `wp_mime_type_icon()` looks
a bit iffy to me and likely to return unexpected results in some
situations, but that should be the subject of another ticket.
I have not addressed the issue of inconsistencies in labelling and
numbering between the PHP and JavaScript views. Again, those are perhaps
better dealt with on a separate ticket.
--
Ticket URL: <https://core.trac.wordpress.org/ticket/30788#comment:6>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform
More information about the wp-trac
mailing list