[wp-trac] [WordPress Trac] #60120: Add cache group to block pattern cache
WordPress Trac
noreply at wordpress.org
Mon Jan 8 23:03:26 UTC 2024
#60120: Add cache group to block pattern cache
--------------------------------------+--------------------------
Reporter: spacedmonkey | Owner: joemcgill
Type: defect (bug) | Status: assigned
Priority: normal | Milestone: 6.5
Component: Themes | Version: 6.4
Severity: normal | Resolution:
Keywords: has-patch has-unit-tests | Focuses: performance
--------------------------------------+--------------------------
Comment (by flixos90):
@spacedmonkey @joemcgill
I'm not sure about the value of conditionally having the new group non-
persistent. In all other places in core we avoid caching entirely based on
`WP_DEVELOPMENT_MODE`. I think changing that warrants a bigger discussion.
I agree with @joemcgill that we need to consider this as part of the
broader solution, though I also think it's fair to go with what you
suggested @spacedmonkey and address only the minimum requirements that
this ticket originally requested as part of the PR.
Given you previously suggested to limit the scope here, I'd say let's
limit the PR to purely introducing the `theme_files` cache group, which
was the only original purpose of this ticket.
We can then address the broader approach with a single PR against #59719.
--
Ticket URL: <https://core.trac.wordpress.org/ticket/60120#comment:14>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform
More information about the wp-trac
mailing list