Changeset 37798 for branches/4.4/src/wp-includes/embed.php

Timestamp:

06/21/2016 02:41:05 PM (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().

Merge of [37708], [37710] and [37729] to the 4.4 branch.

Fixes #36767.

File:

• branches/4.4/src/wp-includes/embed.php (modified) (1 diff)

branches/4.4/src/wp-includes/embed.php

@@ -1046 +1046 @@
     <?php
 }
+
+/**
+ * Filters the oEmbed result before any HTTP requests are made.
+ *
+ * If the URL belongs to the current site, the result is fetched directly instead of
+ * going through the oEmbed discovery process.
+ *
+ * @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 that should be inspected for discovery `<link>` tags.
+ * @param array       $args   oEmbed remote get arguments.
+ * @return null|string The UNSANITIZED (and potentially unsafe) HTML that should be used to embed.
+ *                     Null if the URL does not belong to the current site.
+ */
+function wp_filter_pre_oembed_result( $result, $url, $args ) {
+    $post_id = url_to_postid( $url );
+
+    /** This filter is documented in wp-includes/class-wp-oembed-controller.php */
+    $post_id = apply_filters( 'oembed_request_post_id', $post_id, $url );
+
+    if ( ! $post_id ) {
+        return $result;
+    }
+
+    $width = isset( $args['width'] ) ? $args['width'] : 0;
+
+    $data = get_oembed_response_data( $post_id, $width );
+    $data = _wp_oembed_get_object()->data2html( (object) $data, $url );
+
+    if ( ! $data ) {
+        return $result;
+    }
+
+    return $data;
+}