Skip to:
Content
Pages
Categories
Search
Top
Bottom

*Link to Messages in Email Notifications not working – Error in trunk code?

  • Avatar of quaker-oats
    quaker-oats
    Participant

    @quaker-oats

    This is a followup to a post that I don’t think was resolved: http://buddypress.org/forums/topic/private-message-issue-mulitple-receipents-email-notification-empty-content#post-7103

    Essentially: the link to messages in email notifications do not go to the message thread.

    The reason the links are not working are possibly because of an error in the bp code.

    In the function messages_notification_new_message() (in bp-messages-notifications.php),

    line 15 says:

    $message_link = site_url() . ‘/’ . BP_MEMBERS_SLUG . ‘/’ . $ud->user_login . ‘/messages/view/’ . $message->id;

    This is incorrect. Instead of having $message->id at the end, it should be putting in the *THREAD_ID*.

    If I change this to the thread-id, then the link successfully goes to the message thread. Otherwise it doesn’t work.

    Can someone experienced in the buddypress code please confirm this? Or am I doing something wrong/mis-understanding something?

    Thanks!

Viewing 3 replies - 1 through 3 (of 3 total)
Viewing 3 replies - 1 through 3 (of 3 total)

You must be logged in to reply to this topic.