[buddypress-trac] [BuddyPress Trac] #5333: new_blog_post activity items not deleted when blog post is deleted

[buddypress-trac]

#5333: new_blog_post activity items not deleted when blog post is deleted
--------------------------+--------------------
 Reporter:  henry.wright  |       Owner:
     Type:  defect (bug)  |      Status:  new
 Priority:  normal        |   Milestone:  2.0
Component:  Activity      |     Version:  1.9.1
 Severity:  normal        |  Resolution:
 Keywords:  has-patch     |
--------------------------+--------------------

Comment (by DJPaul):

 Patch incoming. Spent a few hours looking at this.

 * The attached patch works, but probably nicer to integrate into a
 `bp_blogs_catch_published_post()`-type wrapper per Boone's comments.
 * Also correct that someone could trash and untrash, and as Activity
 doesn't have an equivalent "trashed" status, so we either have to leave
 the activity items behind pointing to dead links, or they loose the
 activity item, per this ticket. :) Consensus seems to be that we should
 delete activity when a post is trashed.
 * I discovered when testing that we don't handle Activity items if the
 post goes from published -> draft.

--
Ticket URL: <https://buddypress.trac.wordpress.org/ticket/5333#comment:13>
BuddyPress Trac <http://buddypress.org/>
BuddyPress Trac