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

Timestamp:

08/20/2014 07:31:46 PM (12 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;