[wp-trac] [WordPress Trac] #40977: Widgets: Query param for `loop` added for non-hosted external videos

[WordPress Trac]

#40977: Widgets: Query param for `loop` added for non-hosted external videos
-------------------------------------------------+-------------------------
 Reporter:  westonruter                          |       Owner:
     Type:  defect (bug)                         |  westonruter
 Priority:  normal                               |      Status:  closed
Component:  Widgets                              |   Milestone:  4.8.1
 Severity:  normal                               |     Version:  4.8
 Keywords:  has-patch has-unit-tests fixed-      |  Resolution:  fixed
  major                                          |     Focuses:
-------------------------------------------------+-------------------------
Changes (by westonruter):

 * status:  reopened => closed
 * resolution:   => fixed


Comment:

 In [changeset:"41054"]:
 {{{
 #!CommitTicketReference repository="" revision="41054"
 Media: Restrict appending `loop` parameter to Vimeo URLs specifically and
 not all external URLs in Video widget (via shortcode).

 Fixes issue where Video widgets embedding external files fail to get
 recognized due to the presence of the `loop` param after the video
 filename, even though it has a recognized extension. Regardless, the
 `loop` param is only present to fix a Vimeo issue in ME.js 2.x.

 Merges [40892] into the 4.8 branch.
 Props timmydcrawford.
 Amends [40640].
 See #39686, #39994.
 Fixes #40977 for 4.8.1.
 }}}

--
Ticket URL: <https://core.trac.wordpress.org/ticket/40977#comment:7>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform