[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:  westonruter
     Type:  defect (bug)              |      Status:  closed
 Priority:  normal                    |   Milestone:  4.8.1
Component:  Widgets                   |     Version:  4.8
 Severity:  normal                    |  Resolution:  fixed
 Keywords:  has-patch has-unit-tests  |     Focuses:
--------------------------------------+--------------------------
Changes (by westonruter):

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


Comment:

 In [changeset:"40892"]:
 {{{
 #!CommitTicketReference repository="" revision="40892"
 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.

 Props timmydcrawford.
 Amends [40640].
 See #39686, #39994.
 Fixes #40977.
 }}}

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