Security Alert: 26 BuddyDev accounts compromized in brute force attack.

BuddyDev

Search

Double Notification On Nested Comments – Buddypress Activity Comment Notifier

Tagged: , ,

  • Participant
    Level: Initiated
    Posts: 6
    Fariz on November 4, 2017 at 10:54 pm #11549

    Hello,
    I am using the Buddypress Activity Comment Notifier plugin and noticed that the Blog Author was receiving double notifications when there is a new comment. So after searching this forum I found this function
    that removes the double notification for the blog author.

    
add_filter( 'ac_notifier_skip_notification', 'bp_ac_custom_exclude_comment_activity', 10, 2 );

function bp_ac_custom_exclude_comment_activity( $exclude, $activity ) {
    if ( ! empty( $activity ) && $activity->type =='new_blog_post' && (get_current_user_id() != $activity->user_id))
    {   
        return true;
    }
    return $exclude;
}

    However I am still receiving double notification on replies to user comments.

    Example: User John Posted a comment on a Blog Post and receives a reply on his comment.
    John receives 2 notifications.

    I am trying to remove these double notifications on replies to comments.
    [Additional Information: I am using 2 level of nesting for comments]

    Any help would be greatly appreciated.

    Thanks,
    Fariz

  • Keymaster
    (BuddyDev Team)
    Posts: 24706
    Brajesh Singh on November 4, 2017 at 10:59 pm #11550

    Hi Fariz,
    You most probably don’t need the plugin.

    BuddyPress already implements comment notification now(Not as detailed as the plugin though)

    In the double notification one is coming from BuddyPress and one is from the plugin. That’s why you are seeing it twice.

    Please disable the plugin and see if you really need this plugin.

    If you still do, please let me know and I will assist.

    Regards
    Brajesh

  • Participant
    Level: Initiated
    Posts: 6
    Fariz on November 5, 2017 at 4:30 am #11553

    Hi Brajesh,
    Thank you very much for your help. I disabled the plugin and tried it.
    When a new comment is posted all the users in the comment thread should receive a notification(like facebook),
    but in default buddypress only the blog author receives the comment notification. That is why I am using the Buddypress Activity Comment Notifier to send comment notification to all users in the comment thread.
    The problem arises when the Blog Author replies to my comment, I already receive a notification from Buddypress, but Buddypress Activity Comment Notifier also sends a notification. So I was trying to exclude notification for replies to comment in Buddypress Activity Comment Notifier.

    Thanks,
    Fariz

  • Keymaster
    (BuddyDev Team)
    Posts: 24706
    Brajesh Singh on November 6, 2017 at 9:52 pm #11582

    Hi Fariz,
    Thank you for the details.

    Please allow us a week to revive this plugin. We had abandoned it when BuddyPress included support for the comment notification.

    We will update it within a week and let you know.

    Thank you
    Brajesh

  • Participant
    Level: Initiated
    Posts: 6
    Fariz on November 7, 2017 at 2:51 pm #11600

    That is great. Thank you very much Brajesh.

  • Keymaster
    (BuddyDev Team)
    Posts: 24706
    Brajesh Singh on November 7, 2017 at 7:48 pm #11606

    Thank you for the patience Fariz.

  • Participant
    Level: Initiated
    Posts: 6
    Fariz on November 17, 2017 at 6:14 am #11751

    Hi Brajesh,
    I just wanted to follow up to see if there is any updates for this plugin?
    If not, could you tell how I can fix the above mentioned function to filter out notification for replies to comment, since buddypress by default sends a notification for that. I appreciate your help.

    Thanks,
    Fariz

  • Keymaster
    (BuddyDev Team)
    Posts: 24706
    Brajesh Singh on November 21, 2017 at 3:21 am #11784

    Hi Fariz,
    We will have an update this week for sure. Sorry to keep you waiting.

    Regards
    Brajesh

  • Participant
    Level: Initiated
    Posts: 6
    Fariz on November 21, 2017 at 5:45 am #11797

    Hi Brajesh,
    Not a problem.Thank you for the update.

  • Participant
    Level: Initiated
    Posts: 6
    Fariz on December 2, 2017 at 8:19 pm #12002

    Hi Brajesh,
    Any update on this ?

You must be logged in to reply to this topic.

This topic is: not resolved