Changeset 40641

Timestamp:

05/11/2017 09:23:50 PM (8 years ago)

Author:

westonruter

Message:

Widgets: Utilize WP REST API oEmbed proxy endpoint in media widgets.

Props timmydcrawford.
See #32417, #40450.

Location:

trunk/src/wp-admin/js/widgets

Files:

• media-video-widget.js (modified) (1 diff)
• media-widgets.js (modified) (1 diff)

trunk/src/wp-admin/js/widgets/media-video-widget.js

@@ -112 +112 @@
 
             control.fetchEmbedDfd = jQuery.ajax({
-                url: 'https://noembed.com/embed',
+                url: wp.media.view.settings.oEmbedProxyUrl,
                 data: {
                     url: control.model.get( 'url' ),
                     maxwidth: control.model.get( 'width' ),
-                    maxheight: control.model.get( 'height' )
+                    maxheight: control.model.get( 'height' ),
+                    _wpnonce: wp.media.view.settings.nonce.wpRestApi,
+                    discover: false
                 },
                 type: 'GET',
-                crossDomain: true,
-                dataType: 'json'
+                dataType: 'json',
+                context: control
             });
 

trunk/src/wp-admin/js/widgets/media-widgets.js

@@ -183 +183 @@
 
                         embedLinkView.dfd = $.ajax({
-                            url: 'https://noembed.com/embed', // @todo Replace with core proxy endpoint once committed.
+                            url: wp.media.view.settings.oEmbedProxyUrl,
                             data: {
                                 url: embedLinkView.model.get( 'url' ),
                                 maxwidth: embedLinkView.model.get( 'width' ),
-                                maxheight: embedLinkView.model.get( 'height' )
+                                maxheight: embedLinkView.model.get( 'height' ),
+                                _wpnonce: wp.media.view.settings.nonce.wpRestApi,
+                                discover: false
                             },
                             type: 'GET',
-                            crossDomain: true,
-                            dataType: 'json'
+                            dataType: 'json',
+                            context: embedLinkView
                         });