[buddypress-trac] [BuddyPress Trac] #5130: Synchronizing activity comments to main component

[buddypress-trac]

#5130: Synchronizing activity comments to main component
-------------------------+--------------------
 Reporter:  r-a-y        |       Owner:  r-a-y
     Type:  enhancement  |      Status:  new
 Priority:  normal       |   Milestone:  2.0
Component:  Core         |     Version:  1.2
 Severity:  normal       |  Resolution:
 Keywords:               |
-------------------------+--------------------

Comment (by imath):

 hi r-a-y

 First: very nice work. I like a lot this feature.

 I've tested your patch and this is my feedback :)

 1. If from the WP Admin comments screen i edit a comment, then the
 corresponding activity is not edited, a new one is created. And it's no
 more possible to reply to this activity. We can only delete it and so on
 for the next activity synced from new comments.
 2. If i trash a comment, it's still displayed in the activity stream. If i
 trash a post, the post and the comments disappear. When i untrash the
 post, the activity of this post is there, the comments no more, they are
 deleted. In this particular case the comment_metas
 'bp_activity_comment_id' are not deleted.
 3. In the Activity Admin, the activity of comments are above the activity
 of post. When replying to the activity of the post from this area, ajax
 puts the activity under the one of the post. If i refresh, the activity of
 the comment go on top.
 4. I've tested with a custom post type by filtering
 {{{'bp_blogs_record_post_post_types'}}} and
 {{{'bp_blogs_record_comment_post_types'}}} and it works fine.

 i'm very interested as it will have an impact on #3460 :)

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