[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