Changes between Initial Version and Version 1 of Ticket #5130, comment 31
- Timestamp:
- 03/28/2014 06:41:58 PM (12 years ago)
Legend:
- Unmodified
- Added
- Removed
- Modified
-
Ticket #5130, comment 31
initial v1 1 1 > To be honest, I'm still not clear on why we'd want this to be true, but I think your solution is fine for 2.0. Shouldn't we also be doing something in bp_blogs_sync_delete_post_comments_on_post_delete()? You have a $force_delete param but I can't see where it'd ever be used (you're hooked to 'bp_blogs_before_remove_post'. 2 2 3 Sorry, forgot to remove that part of the code from the patch. Was experimenting with trash / untrash post status changes in `bp_blogs_ transition_activity_status()` and how that handles comments and activity items related to comments.3 Sorry, forgot to remove that part of the code from the patch. Was experimenting with trash / untrash post status changes in `bp_blogs_catch_transition_post_status()` and how that handles comments and activity items related to comments. 4 4 5 5 There are too many things required to get this working for 2.0-beta, so I decided to hold off.
![(please configure the [header_logo] section in trac.ini)](/chrome/site/your_project_logo.png)