WordPress.org

Make WordPress Core

Opened 3 years ago

Last modified 2 years ago

#40363 assigned defect (bug)

Remove `current_blog_` cache invalidation from `clean_blog_cache()`

Reported by: flixos90 Owned by: spacedmonkey
Milestone: Awaiting Review Priority: normal
Severity: normal Version:
Component: Networks and Sites Keywords: good-first-bug has-patch
Focuses: multisite Cc:
PR Number:

Description

clean_blog_cache() currently invalidates keys current_blog_{$blog->domain} and current_blog_{$blog->domain}_{$blog->path} of the site-options group. As far as I can see those keys are not used anywhere in Core (and have not been used for quite a while), so we should be able to remove them from that function.

Related: #40201 (more elaborate improvements of clean_blog_cache())

Attachments (1)

40363.patch​ (628 bytes) - added by spacedmonkey 3 years ago.

Download all attachments as: .zip

Change History (9)

@spacedmonkey
3 years ago

#1 @spacedmonkey
3 years ago

I checked and these cache keys haven't been used in core for sometime. It should be safe to remove them.

#2 @thamaraiselvam
3 years ago

I am also confirming that these keys are not used anywhere in the core. better to remove them.

#3 @thamaraiselvam
3 years ago

  • Keywords has-patch added; needs-patch removed

This ticket was mentioned in ​Slack in #core-multisite by spacedmonkey. ​View the logs.
3 years ago

This ticket was mentioned in ​Slack in #core-multisite by flixos90. ​View the logs.
3 years ago

#6 @DrewAPicture
2 years ago

  • Owner set to spacedmonkey
  • Status changed from new to assigned

Assigning ownership to mark the good-first-bug as "claimed".

This ticket was mentioned in ​Slack in #core-multisite by flixos90. ​View the logs.
2 years ago

#8 @flixos90
2 years ago

@jeremyfelt @johnjamesjacoby I am not sure, but I think I remember there were some concerns about removing these?

Note: See TracTickets for help on using tickets.