Changeset 43810 for branches/5.0/src/wp-includes/class-wp-oembed-controller.php

Timestamp:

10/23/2018 05:47:28 PM (6 years ago)

Author:

danielbachhuber

Message:

Embeds: Filter HTML response in oEmbed proxy controller.

Adapts the response from WP_oEmbed_Controller::get_proxy_item() so that the response is correctly filtered and embeds work properly in JavaSccript editors. Introduces new get_oembed_response_data_for_url() function for preparing internal oEmbed responses.

Props danielbachhuber, imath, swissspidy.
Fixes #45142.

File:

• branches/5.0/src/wp-includes/class-wp-oembed-controller.php (modified) (2 diffs)

branches/5.0/src/wp-includes/class-wp-oembed-controller.php

@@ -174 +174 @@
         }
 
+        // Short-circuit process for URLs belonging to the current site.
+        $data = get_oembed_response_data_for_url( $url, $args );
+
+        if ( $data ) {
+            return $data;
+        }
+
         $data = _wp_oembed_get_object()->get_data( $url, $args );
 
@@ -179 +186 @@
             return new WP_Error( 'oembed_invalid_url', get_status_header_desc( 404 ), array( 'status' => 404 ) );
         }
+
+        /** This filter is documented in wp-includes/class-oembed.php */
+        $data->html = apply_filters( 'oembed_result', _wp_oembed_get_object()->data2html( (object) $data, $url ), $url, $args );
 
         /**