`bp_activity_comments` cache should purge when an any child `activity_comment` is marked as spam

[oztaser]

We need to purge the parent activity bp_activity_comments cache when an activity comment marked as spam.

Steps to Reproduce:

1. Enable object cache
2. Add an activity comment to any activity
3. Go to activity list page on admin
4. Find an activity comment and marked as a spam
5. Go back to activity page and see activity comment is still there

[imath]

Hi @oztaser

I'm wondering, shouldn't we purge cache for any type of activity marked as spam ? Why restricting this to activity comment ?

[oztaser]

Hi @imath,

What I suggested here in this ticket is purging activity_comments cache. If you add activity comment to A activity and then mark as spam your activity comment, activity_comments cache of A activity needs to purge.

Based on your point we're already purging caches when activity marked as spam, If I don't missing something.

I took an another look at my patch, It does't look perfect :). We may purge $activity->item_id cache in bp_activity_mark_as_spam function.

[imath]

Hi @oztaser

Thanks for your explanations. I think we need to review how the bp_activity and bp_activity_comments cache are purged. Imho, when an activity is marked as spam/ham we should probably also purge the bp_activity cache if the activity is not an activity comment.

I also think we should take your patch in account, but avoid checking for is_spam as a comment content can simply be updated by the Administrator from the Activity Administration screen.

I'd feel more secure if we could build some unit tests about these purges.

I'd also like to get feedback about it from @r-a-y as he's been working on this area in the past.

[oztaser]

Hi @imath,

Thanks for your work. I totally agree with you and your patch looks good to me.

[imath]

8296.3.patch is including the Unit tests. I finally think it's safest to leave the bp_activity cache when an activity has been spam/ham.

[imath]

In 13149:

Improve the bp_activity_comments cache management

This cache should also be cleared when:

• an activity comment has been marked as spam/ham.
• an activity comment has been updated.

Props oztaser

Fixes #8296