Skip to:
Content

Opened 3 years ago

Closed 3 years ago

#2855 closed defect (fixed)

Clear topic favoriters cache when user adds/removes favorite

Reported by: thebrandonallen Owned by: netweb
Milestone: 2.6 Priority: normal
Severity: normal Version: 2.5.8
Component: API - Cache Keywords: has-patch
Cc:

Description

Started writing tests for #2488 and realized that I couldn't reliably test the removing of favorites and subscriptions without first testing that the functions to remove favorites and subscriptions actually work as expected. They do, except for cache clearing when adding/removing favorites.

Attachments (1)

2855.1.patch (25.9 KB) - added by thebrandonallen 3 years ago.

Download all attachments as: .zip

Change History (5)

#2 @johnjamesjacoby
3 years ago

In 5926:

Favorites: Purge cache when adding/removing a favorite.

This changeset fixes a bug where cache was not busted, causing incorrect results to be returned & displayed.

Props thebrandonallen. See #2855.

#3 @johnjamesjacoby
3 years ago

I couldn't get the remaining portion of the patch to apply to the Subversion repository, so they'll need to be applied separately later.

#4 @netweb
3 years ago

  • Owner set to netweb
  • Resolution set to fixed
  • Status changed from new to closed

In 5928:

Tests: User subscription and user favorites tests in BBP_Tests_Users_Functions_Subscriptions and BBP_Tests_Users_Functions_Favorites

Props thebrandonallen. Fixes #2855.

Note: See TracTickets for help on using tickets.