[wp-trac] [WordPress Trac] #49412: Store filesize media metadata

WordPress Trac noreply at wordpress.org
Wed Oct 28 13:36:13 UTC 2020


#49412: Store filesize media metadata
-----------------------------+-----------------------------
 Reporter:  Cybr             |       Owner:  (none)
     Type:  feature request  |      Status:  new
 Priority:  normal           |   Milestone:  Future Release
Component:  Media            |     Version:
 Severity:  normal           |  Resolution:
 Keywords:  has-patch        |     Focuses:  performance
-----------------------------+-----------------------------

Comment (by spacedmonkey):

 I just reviewed @johnwatkins0 patch. It look great to me.

 The place that I feel was missed was the [https://github.com/WordPress
 /wordpress-develop/blob/7e4debe75c07cc7f97bb40bad451cd9469a409da/src/wp-
 admin/includes/image.php#L183-L212 _wp_image_meta_replace_original]
 function.

 Otherwise, this patch is good to me.

 As I noted in #51650

 > Many popular plugins like Yoast, ACF and Jetpack, use the function
 filesize to get the file size of a attachment.

 So I would love to see this land in core.

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


More information about the wp-trac mailing list