BuddyDev

Search

BP Activity Comment Notifier. Nested comments notification owner is wrong

  • Participant
    Jesus Iniesta on #35014

    When a user comments on an existing comment, the notification refers to the owner of the comment as the owner of the post, which is wrong.

    To change this behaviour seems as simple as replacing the existing if/else statement in bp-activity-comment-notifier/core/functions.php:103
    `php
    if ($total_items > 1) {
    $users = ac_notifier_find_involved_persons($activity_id);
    } else {
    $activity = new BP_Activity_Activity($secondary_item_id);
    $users = array(0 => $activity->user_id);
    }
    `

    With:

    `php
    if ($total_items > 1) {
    $users = ac_notifier_find_involved_persons($activity_id);
    } else {
    $activity = new BP_Activity_Activity($secondary_item_id);
    $users = array(0 => $activity->user_id);
    if ($activity->secondary_item_id > 0) {
    $activity = new BP_Activity_Activity($activity->secondary_item_id);
    }
    }
    `

You must be logged in to reply to this topic.

This topic is: not resolved