[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