Changeset 40628 for trunk/src/wp-includes/class-oembed.php

Timestamp:

05/11/2017 06:18:00 PM (9 years ago)

Author:

jnylen0

Message:

REST API: Add endpoint for proxying requests to external oEmbed providers.

This endpoint is a prerequisite for the media widgets work (see ​https://github.com/xwp/wp-core-media-widgets).

Also use the new endpoint in the media modal instead of the parse-embed AJAX action.

Props westonruter, timmydcrawford, swissspidy, jnylen0.
Fixes #40450.

File:

• trunk/src/wp-includes/class-oembed.php (modified) (3 diffs)

trunk/src/wp-includes/class-oembed.php

@@ -320 +320 @@
 
         /**
+         * Takes a URL and attempts to return the oEmbed data.
+         *
+         * @see WP_oEmbed::fetch()
+         *
+         * @since 4.8.0
+         * @access public
+         *
+         * @param string       $url  The URL to the content that should be attempted to be embedded.
+         * @param array|string $args Optional. Arguments, usually passed from a shortcode. Default empty.
+         * @return false|object False on failure, otherwise the result in the form of an object.
+         */
+        public function get_data( $url, $args = '' ) {
+                $args = wp_parse_args( $args );
+
+                $provider = $this->get_provider( $url, $args );
+
+                if ( ! $provider ) {
+                        return false;
+                }
+
+                $data = $this->fetch( $provider, $url, $args );
+
+                if ( false === $data ) {
+                        return false;
+                }
+
+                return $data;
+        }
+
+        /**
          * The do-it-all function that takes a URL and attempts to return the HTML.
          *
@@ -333 +363 @@
          */
         public function get_html( $url, $args = '' ) {
-                $args = wp_parse_args( $args );
-
                 /**
                  * Filters the oEmbed result before any HTTP requests are made.
@@ -356 +384 @@
                 }
 
-                $provider = $this->get_provider( $url, $args );
-
-                if ( ! $provider || false === $data = $this->fetch( $provider, $url, $args ) ) {
+                $data = $this->get_data( $url, $args );
+
+                if ( false === $data ) {
                         return false;
                 }