[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