Object caching is incompatible with mutative the_posts filters.

[Antiarc]

query.php runs something like:

1. Get the posts
2. Run the_posts filters
3. Save $posts in object cache

The problem is that when you go to edit a post, get_post() checks the object cache for a post before pulling it from the DB. Thus, presume the following:

1. We pull a post, with the content "foo".
2. We run a the_posts hook, maybe with a regex that transforms "foo" into "foobar".
3. $posts are saved in the object cache
4. We go to edit a post. Post is pulled from the object cache, as "foobar" - not what the user entered into their post.
5. We save the post without any editing. Post gets saved as "foobar".
6. The post is pulled from the DB and the_posts is run. Our regex transforms it to "foobarbar", etc etc.

I know that transformative actions are generally done on the_content, but in this particular case, I absolutely have to be doing batched operations on all posts pulled (it's sending content off to a remote server for processing, and running that on the_content would generate N requests per page, rather than 1 request per page, incurring massive slowdown). The docs do not forbid transformation by the_posts, and I can't find a more appropriate hook to use, nor is there a hook to invalidate the contents of the object cache before editing happens.

This used to be correct, but was changed in r4517 - the commit message there is kind of bogus. I've attached a patch with the fix.

[Denis-de-Bernardy]

I imagine the goal was to also cache the results of that filter, so as to not run it twice.

the filter is only used once in WP, but there almost certainly are plugins that rely on it in a way or another.

imo, patch is a good idea, unless there's a very popular plugin whose behavior is changed by this.

[ryan]

Looking at plugins to judge the impact. The current behavior is not good behavior, but it has been that way for awhile and bugs have a way of becoming interface.

[Denis-de-Bernardy]

maybe we should add an extra hook in there instead?

[Denis-de-Bernardy]

will this one get committed, or will it rot for a few years?

[Denis-de-Bernardy]

patch still applies clean

[Denis-de-Bernardy]

I've uploaded an improved patch that ensures posts are sanitized before they're cached.

[Denis-de-Bernardy]

That being said, I'm 80% sure that this should get a wontfix: the more I look at it (I'm working on a query caching plugin, which could actually make use of that hook if WP had a few more in there), the more convinced I am that the current behavior is the correct one on this front.

I'm pretty certain that you should be using the loop_start or the the_post filter (both were introduced in 2.8) for whatever you are trying to do. Or maybe even hook into the wp hook directly...

[ryan]

I've gone back and forth on whether this should change. It's going to break certain scenarios either way. Let's call it wontfix.