WordPress.org

Make WordPress Core

Opened 11 months ago

Last modified 7 months 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:

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 11 months ago.

Download all attachments as: .zip

Change History (9)

#1 @spacedmonkey
11 months ago

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

#2 @thamaraiselvam
11 months ago

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

#3 @thamaraiselvam
11 months ago

  • Keywords has-patch added; needs-patch removed

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


11 months ago

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


11 months ago

#6 @DrewAPicture
8 months 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.


7 months ago

#8 @flixos90
7 months 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.