[wp-trac] [WordPress Trac] #36905: Add caching to get_page_by_title
WordPress Trac
noreply at wordpress.org
Sun Dec 16 01:54:32 UTC 2018
#36905: Add caching to get_page_by_title
-------------------------------------------------+-------------------------
Reporter: spacedmonkey | Owner: igmoweb
Type: enhancement | Status: assigned
Priority: normal | Milestone: Future
| Release
Component: Posts, Post Types | Version: 2.1
Severity: normal | Resolution:
Keywords: has-patch good-first-bug has-unit- | Focuses:
tests | performance
-------------------------------------------------+-------------------------
Comment (by spacedmonkey):
@igmoweb @pcfreak30 Looking at your patches, I have one simple question,
how does this fix the issue? Using WP_Query is nice, but WP_Query isn't
have any query caching out of the box. Even if are you using
[https://github.com/Automattic/advanced-post-cache advanced-post-cache] or
[https://github.com/TimeIncUK/enhanced-post-cache enhanced-post-cache]
neither of which, cache queries with 'fields' => 'ids' set.
See that #36711 is basically the same code and @boonebgorges merged that.
We should go back to my original patch. This will likely need a refresh
after all coding standards changes.
--
Ticket URL: <https://core.trac.wordpress.org/ticket/36905#comment:10>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform
More information about the wp-trac
mailing list