[wp-trac] [WordPress Trac] #24330: When adding the post_formats_compat the_content filter a post ID argument should be included

WordPress Trac noreply at wordpress.org
Thu May 16 15:32:27 UTC 2013


#24330: When adding the post_formats_compat the_content filter a post ID argument
should be included
--------------------------+--------------------
 Reporter:  gcorne        |       Owner:
     Type:  defect (bug)  |      Status:  new
 Priority:  normal        |   Milestone:  3.6
Component:  Post Formats  |     Version:  trunk
 Severity:  normal        |  Resolution:
 Keywords:                |
--------------------------+--------------------

Comment (by ryan):

 I'll use this ticket for some quick notes on the guts of post formats.

 It'd be nice if $format_pages is only set when absolutely necessary, such
 as when $post->split_content is set. Even better would be for this global
 to not exist. We have enough problems with $pages as it is. Perhaps we run
 with the patch here after all and extend it to make the_content() and
 the_remaining_content() independent of the globals in setup_postdata().

 $post->split_content and $post->format_content[ $cache_key ] are bummers.
 If we're going to decorate WP_Post with these perhaps they should be
 declared as vars and documented with phpdoc. Also, are these getting into
 the object cache? Should they?

-- 
Ticket URL: <http://core.trac.wordpress.org/ticket/24330#comment:10>
WordPress Trac <http://core.trac.wordpress.org/>
WordPress blogging software


More information about the wp-trac mailing list