Make WordPress Core

Opened 8 years ago

Closed 8 years ago

#36203 closed enhancement (fixed)

clean_blog_cache needs a hook

Reported by: spacedmonkey's profile spacedmonkey Owned by: jeremyfelt's profile jeremyfelt
Milestone: 4.6 Priority: normal
Severity: normal Version:
Component: Cache API Keywords: has-patch
Focuses: multisite Cc:

Description

Unlike other cache clear functions, clean_blog_cache doesn't have a hook.

Attachments (1)

36203.patch (762 bytes) - added by spacedmonkey 8 years ago.

Download all attachments as: .zip

Change History (5)

@spacedmonkey
8 years ago

#1 follow-up: @spacedmonkey
8 years ago

I have named it the hook clean_site_cache. I know the function is called clean_blog_cache, but want to keep inline with multisite naming.

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


8 years ago

#3 in reply to: ↑ 1 @jeremyfelt
8 years ago

  • Keywords has-patch added
  • Milestone changed from Awaiting Review to 4.6
  • Owner set to jeremyfelt
  • Status changed from new to accepted

Replying to spacedmonkey:

I have named it the hook clean_site_cache. I know the function is called clean_blog_cache, but want to keep inline with multisite naming.

I think that's the right call. We don't have a clean_site_cache() to confuse with and we would likely introduce a clean_network_cache() instead if it was ever needed.

#4 @jeremyfelt
8 years ago

  • Resolution set to fixed
  • Status changed from accepted to closed

In 37204:

Multisite: Fire a clean_site_cache action when a site's cache is cleared

Props spacedmonkey.
Fixes #36203.

Note: See TracTickets for help on using tickets.