WordPress.org

Make WordPress Core

Changes between Initial Version and Version 1 of Ticket #41201, comment 22


Ignore:
Timestamp:
07/24/2017 05:29:15 PM (3 years ago)
Author:
timmydcrawford
Comment:

Legend:

Unmodified
Added
Removed
Modified
  • Ticket #41201, comment 22

    initial v1  
    1616The URL is filterable via https://core.trac.wordpress.org/browser/trunk/src/wp-includes/class-oembed.php?#L505 so one route would be to watch for these embed URLs and transform them to the share url format, or standard YouTube url format.  But to me that kind of feels hackish.
    1717
    18 This is just like what happens in https://opengrok.a8c.com/source/xref/trunk/wp-includes/embed.php#230 as @swissspidy pointed out in slack.
     18This is just like what happens in as @swissspidy pointed out in slack.
    1919
    2020So after all of that, I'm kind of thinking that since the embed URLs are not supported by YouTube in the context of their oembed endpoint, not certain if we should support it either.