[wp-trac] [WordPress Trac] #35218: Parse the creation date out of uploaded videos

WordPress Trac noreply at wordpress.org
Wed Oct 4 08:51:05 UTC 2017


#35218: Parse the creation date out of uploaded videos
--------------------------------------+---------------------------
 Reporter:  stevegrunwell             |       Owner:  mikeschroder
     Type:  enhancement               |      Status:  reviewing
 Priority:  normal                    |   Milestone:  4.9
Component:  Media                     |     Version:
 Severity:  normal                    |  Resolution:
 Keywords:  has-patch has-unit-tests  |     Focuses:
--------------------------------------+---------------------------

Comment (by mikeschroder):

 Chatted with @joemcgill about this tonight.
 Three new patches, can commit one of them in the morning (or feel free to
 do it, @joemcgill if it needs to happen first thing, and you're working
 first).

 I'm okay with all of these approaches, but prefer the last.

 - [attachment:35218.6.diff] is basically an update. Only create meta if
 there is a creation date and only rely on small* test media files.

 - [attachment:video_meta.diff] places the new data (creation timestamp)
 into a `video_meta` array in meta, much like `image_meta` on images.

 - [attachment:video_meta_expanded_filter.diff] like the diff above, but
 also swaps the proposed filter for `wp_read_video_metadata`, which closely
 matches `wp_read_image_metadata`.

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


More information about the wp-trac mailing list