![(please configure the [header_logo] section in trac.ini)](/chrome/site/your_project_logo.png){kind=logo}
#42070 closed enhancement (fixed)
Remove networks_have_paths cache key
| Reported by: | spacedmonkey | Owned by: | spacedmonkey |
|---|---|---|---|
| Priority: | normal | Milestone: | 6.0 |
| Component: | Networks and Sites | Version: | 4.6 |
| Severity: | normal | Keywords: | has-patch commit |
| Cc: | Focuses: | multisite |
Description
The cache key networks_have_paths is used in bootstrap process to load networks. It caches values stored in from a WP_Network_Query query. WP_Network_Query already has caching, make this cache pointless.
Change History (10)
This ticket was mentioned in PR #2504 on WordPress/wordpress-develop by uday-kokitkar.
4 years ago
#2
- Keywords has-patch added
This ticket was mentioned in PR #2504 on WordPress/wordpress-develop by uday-kokitkar.
4 years ago
#3
Trac ticket: https://core.trac.wordpress.org/ticket/42070
This ticket was mentioned in Slack in #core-multisite by spacedmonkey. View the logs.
4 years ago
#5
@
4 years ago
The PR at #2504 looks good to go. I think this should be committed and be part of 6.0.
4 years ago
#8
The only reservation I can dream up with removing this, is the potential for breakage for sites manually setting this cache key inside of a custom sunrise.php.
The only reason they would do that is specifically to get ahead of and around this query.
peterwilsoncc commented on PR #2504:
4 years ago
#10
Committed in https://core.trac.wordpress.org/changeset/53083
Trac ticket: https://core.trac.wordpress.org/ticket/42070