Skip to:
Content

Opened 3 years ago

Closed 3 years ago

#6062 closed defect (bug) (fixed)

Total blog count cache not invalidated when a site is deleted

Reported by: r-a-y Owned by: r-a-y
Milestone: 2.2 Priority: normal
Severity: normal Version:
Component: Blogs Keywords: has-patch commit
Cc:

Description

When a site is deleted, the total blog count cache is not invalidated.

Attached patch addresses this.

I've also chosen to mark bp_blogs_format_clear_blog_cache() as deprecated because it's not a function that will likely be used by developers.

Let me know what you think.

Attachments (2)

6062.01.patch (2.6 KB) - added by r-a-y 3 years ago.
6062.02.patch (3.5 KB) - added by r-a-y 3 years ago.
02.patch properly deprecates bp_blogs_format_clear_blog_cache() and tweaks bp_blogs_clear_blog_object_cache() to check for the user ID.

Download all attachments as: .zip

Change History (5)

@r-a-y
3 years ago

#1 @johnjamesjacoby
3 years ago

Patch looks good. Nice fix.

@r-a-y
3 years ago

02.patch properly deprecates bp_blogs_format_clear_blog_cache() and tweaks bp_blogs_clear_blog_object_cache() to check for the user ID.

#2 @DJPaul
3 years ago

  • Keywords commit added

Looks good

#3 @r-a-y
3 years ago

  • Owner set to r-a-y
  • Resolution set to fixed
  • Status changed from new to closed

In 9251:

Blogs: Invalidate total blog count when site is deleted.

This commit:

  • Uses bp_blogs_clear_blog_object_cache() to invalidate the total blog count after a site is deleted and when a site is created.
  • Deprecates bp_blogs_format_clear_blog_cache(), a function that isn't used internally as of this commit, for v2.2.0.
  • Adds a unit test.

Fixes #6062.

Note: See TracTickets for help on using tickets.