[wp-trac] [WordPress Trac] #37363: Instance query argument breaks YouTube URL parsing in MediaElement.js

[WordPress Trac]

#37363: Instance query argument breaks YouTube URL parsing in MediaElement.js
--------------------------+----------------------------------
 Reporter:  ocean90       |      Owner:
     Type:  defect (bug)  |     Status:  new
 Priority:  normal        |  Milestone:  Awaiting Review
Component:  Media         |    Version:
 Severity:  normal        |   Keywords:  needs-patch upstream
  Focuses:                |
--------------------------+----------------------------------
 See [comment:ticket:36759:8]:

 Works: `[video src="https://youtu.be/mhnuES3Zgr4"]`
 Broken: `[video src="https://www.youtube.com/watch?v=mhnuES3Zgr4"]`

 That's because `wp_video_shortcode()` appends an
 [https://core.trac.wordpress.org/browser/trunk/src/wp-
 includes/media.php?marks=2300&rev=38052#L2292 instance argument to the
 original URL], which results in
 `http://www.youtube.com/watch?v=mhnuES3Zgr4&_=2`.
 This extra query argument breaks
 [https://github.com/johndyer/mediaelement/blob/1b5d543c8bc35a9f9df947a44440b221904fd823/src/js
 /me-shim.js#L597 this line] in MediaElement.js.

 I'm going to report this upstream as well.

--
Ticket URL: <https://core.trac.wordpress.org/ticket/37363>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform