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

Timestamp:

06/15/2016 11:22:52 AM (8 years ago)

Author:

swissspidy

Message:

Embeds: Improve performance when embedding a post from the current site.

When the post being embedded is from the same site, there's no reason to do an HTTP request for it. The data can be fetched directly using get_oembed_response_data().

Fixes #36767 for trunk.

File:

• trunk/src/wp-includes/class-oembed.php (modified) (1 diff)

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

@@ -316 +316 @@
      */
     public function get_html( $url, $args = '' ) {
+        /**
+         * Filters the oEmbed result before any HTTP requests are made.
+         *
+         * This allows one to short-circuit the default logic, perhaps by
+         * replacing it with a routine that is more optimal for your setup.
+         *
+         * Passing a non-null value to the filter will effectively short-circuit retrieval,
+         * returning the passed value instead.
+         *
+         * @since 4.5.3
+         *
+         * @param null|string $result The UNSANITIZED (and potentially unsafe) HTML that should be used to embed. Default null.
+         * @param string      $url    The URL to the content that should be attempted to be embedded.
+         * @param array       $args   Optional. Arguments, usually passed from a shortcode. Default empty.
+         */
+        $pre = apply_filters( 'pre_oembed_result', null, $url, $args );
+
+        if ( null !== $pre ) {
+            return $pre;
+        }
+
         $provider = $this->get_provider( $url, $args );
 
-        if ( !$provider || false === $data = $this->fetch( $provider, $url, $args ) )
-            return false;
+        if ( ! $provider || false === $data = $this->fetch( $provider, $url, $args ) ) {
+            return false;
+        }
 
         /**