Changeset 29557 for trunk/src/wp-includes/class-wp-embed.php

Timestamp:

08/20/2014 07:31:46 PM (11 years ago)

Author:

SergeyBiryukov

Message:

Don't limit WP_Embed::cache_oembed() to posts and pages.

props bordoni, helen.
fixes #28639.

File:

• trunk/src/wp-includes/class-wp-embed.php (modified) (2 diffs)

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

@@ -294 +294 @@
         $post = get_post( $post_ID );
 
-        $post_types = array( 'post', 'page' );
+        $post_types = get_post_types( array( 'show_ui' => true ) );
         /**
          * Filter the array of post types to cache oEmbed results for.
@@ -300 +300 @@
          * @since 2.9.0
          *
-         * @param array $post_types Array of post types to cache oEmbed results for. Default 'post', 'page'.
+         * @param array $post_types Array of post types to cache oEmbed results for. Defaults to post types with `show_ui` set to true.
          */
-        if ( empty($post->ID) || !in_array( $post->post_type, apply_filters( 'embed_cache_oembed_types', $post_types ) ) )
+        if ( empty( $post->ID ) || ! in_array( $post->post_type, apply_filters( 'embed_cache_oembed_types', $post_types ) ) ){
             return;
+        }
 
         // Trigger a caching
-        if ( !empty($post->post_content) ) {
+        if ( ! empty( $post->post_content ) ) {
             $this->post_ID = $post->ID;
             $this->usecache = false;