Opened 16 months ago
Last modified 3 months ago
#59596 new enhancement
Cache file sizes of core block style files.
Reported by: |
|
Owned by: | |
---|---|---|---|
Milestone: | Future Release | Priority: | normal |
Severity: | normal | Version: | 6.3 |
Component: | Editor | Keywords: | has-patch dev-feedback |
Focuses: | performance | Cc: |
Description
Introduced in #58528 / [56044], there is now cache for file lookups for core block styles. This was to save file lookups and improve server response time. However, when styles are processed in wp_maybe_inline_styles
a file lookup is done using wp_filesize
to get the file size and if the file exists. These lookup can be expensive. As there is already an existing transient cache, reuse that to also cache the file size of the files.
Change History (6)
This ticket was mentioned in PR #5459 on WordPress/wordpress-develop by @spacedmonkey.
16 months ago
#1
- Keywords has-patch added
This ticket was mentioned in PR #5459 on WordPress/wordpress-develop by @spacedmonkey.
16 months ago
#2
- Keywords has-patch added
Trac ticket: https://core.trac.wordpress.org/ticket/59596
This ticket was mentioned in Slack in #core-performance by mukeshpanchal27. View the logs.
16 months ago
#4
@
16 months ago
- Keywords dev-feedback added
Thanks @spacedmonkey. Conceptually, this idea makes sense. It would be great if someone could do an initial analysis of the current performance cost of these wp_filesize
lookups prior to us getting into the implementation details, in order to help prioritize this work appropriately.
Trac ticket: https://core.trac.wordpress.org/ticket/59596