Skip to:
Content

BuddyPress.org

Opened 12 years ago

Closed 12 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
Priority: normal Milestone: 2.2
Component: Blogs Version:
Severity: normal 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 12 years ago.
6062.02.patch​ (3.5 KB ) - added by r-a-y 12 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
12 years ago

#1 @johnjamesjacoby
12 years ago

Patch looks good. Nice fix.

@r-a-y
12 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
12 years ago

  • Keywords commit added

Looks good

#3 @r-a-y
12 years ago

  • Owner set to r-a-y
  • Resolution → fixed
  • Status new → 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.