From 7f3deb9988ead4bc6329f8359fb01d4a64c3119b Mon Sep 17 00:00:00 2001 From: John Blackbourn Date: Fri, 8 Sep 2017 15:00:33 +0000 Subject: [PATCH] Embeds: Remove support for oEmbedded videos from Vine. * Vine effectively shut down in December 2016. * The oEmbed endpoint no longer works and returns a 404. * Developer documentation for Vine has been removed. Fixes #41817 git-svn-id: https://develop.svn.wordpress.org/trunk@41347 602fd350-edb4-49c9-b593-d223f7449a82 --- src/wp-admin/includes/class-wp-press-this.php | 5 +---- src/wp-includes/class-oembed.php | 3 +-- 2 files changed, 2 insertions(+), 6 deletions(-) diff --git a/src/wp-admin/includes/class-wp-press-this.php b/src/wp-admin/includes/class-wp-press-this.php index 1a30e4104d..7f268ea028 100644 --- a/src/wp-admin/includes/class-wp-press-this.php +++ b/src/wp-admin/includes/class-wp-press-this.php @@ -459,7 +459,7 @@ class WP_Press_This { * Limit embed source URLs to specific providers. * * Not all core oEmbed providers are supported. Supported providers include YouTube, Vimeo, - * Vine, Daily Motion, SoundCloud, and Twitter. + * Daily Motion, SoundCloud, and Twitter. * * @ignore * @since 4.2.0 @@ -482,9 +482,6 @@ class WP_Press_This { } else if ( preg_match( '!//vimeo\.com/moogaloop\.swf\?clip_id=([\d]+)$!i', $src, $src_matches ) ) { // Embedded Vimeo Flash videos $src = 'https://vimeo.com/' . (int) $src_matches[1]; - } else if ( preg_match( '!//vine\.co/v/([^/]+)/embed!i', $src, $src_matches ) ) { - // Embedded Vine videos - $src = 'https://vine.co/v/' . $src_matches[1]; } else if ( preg_match( '!//(www\.)?dailymotion\.com/embed/video/([^/?]+)([/?].+)?!i', $src, $src_matches ) ) { // Embedded Daily Motion videos $src = 'https://www.dailymotion.com/video/' . $src_matches[2]; diff --git a/src/wp-includes/class-oembed.php b/src/wp-includes/class-oembed.php index 467b5e5968..48beac3047 100644 --- a/src/wp-includes/class-oembed.php +++ b/src/wp-includes/class-oembed.php @@ -74,7 +74,6 @@ class WP_oEmbed { '#https?://(www\.)?twitter\.com/\w{1,15}/lists/.*#i' => array( 'https://publish.twitter.com/oembed', true ), '#https?://(www\.)?twitter\.com/\w{1,15}/timelines/.*#i' => array( 'https://publish.twitter.com/oembed', true ), '#https?://(www\.)?twitter\.com/i/moments/.*#i' => array( 'https://publish.twitter.com/oembed', true ), - '#https?://vine\.co/v/.*#i' => array( 'https://vine.co/oembed.{format}', true ), '#https?://(www\.)?soundcloud\.com/.*#i' => array( 'https://soundcloud.com/oembed', true ), '#https?://(.+?\.)?slideshare\.net/.*#i' => array( 'https://www.slideshare.net/api/oembed/2', true ), '#https?://(www\.)?instagr(\.am|am\.com)/p/.*#i' => array( 'https://api.instagram.com/oembed', true ), @@ -163,7 +162,6 @@ class WP_oEmbed { * | Polldaddy | poll.fm | Yes | 4.0.0 | * | TED | ted.com | Yes | 4.0.0 | * | YouTube | youtube.com/playlist | Yes | 4.0.0 | - * | Vine | vine.co | Yes | 4.1.0 | * | Tumblr | tumblr.com | Yes | 4.2.0 | * | Kickstarter | kickstarter.com | Yes | 4.2.0 | * | Kickstarter | kck.st | Yes | 4.2.0 | @@ -190,6 +188,7 @@ class WP_oEmbed { * | Blip | blip.tv | No | 2.9.0 | 4.4.0 | * | Rdio | rdio.com | Yes | 3.6.0 | 4.4.1 | * | Rdio | rd.io | Yes | 3.6.0 | 4.4.1 | + * | Vine | vine.co | Yes | 4.1.0 | 4.9.0 | * * @see wp_oembed_add_provider() *