Changeset 40847 for trunk/src/wp-includes/media.php

Timestamp:

05/26/2017 11:09:42 PM (8 years ago)

Author:

westonruter

Message:

Widgets: Normalize YouTube and Vimeo URLs in video shortcode (primarily for Video widget) to work around ME.js 2.22 bug.

Props timmydcrawford, jnylen0, westonruter.
See #32417, #39994.
Fixes #40866.

File:

• trunk/src/wp-includes/media.php (modified) (1 diff)

trunk/src/wp-includes/media.php

@@ -2547 +2547 @@
     }
 
+    // Mediaelement has issues with some URL formats for Vimeo and YouTube, so
+    // update the URL to prevent the ME.js player from breaking.
+    if ( 'mediaelement' === $library ) {
+        if ( $is_youtube ) {
+            // Remove `feature` query arg and force SSL - see #40866.
+            $atts['src'] = remove_query_arg( 'feature', $atts['src'] );
+            $atts['src'] = set_url_scheme( $atts['src'], 'https' );
+        } elseif ( $is_vimeo ) {
+            // Remove all query arguments and force SSL - see #40866.
+            $parsed_vimeo_url = wp_parse_url( $atts['src'] );
+            $atts['src'] = 'https://' . $parsed_vimeo_url['host'] . $parsed_vimeo_url['path'];
+        }
+    }
+
     /**
      * Filters the class attribute for the video shortcode output container.