Skip to:
Content

BuddyPress.org

Opened 7 years ago

Closed 7 years ago

#4597 closed defect (bug) (fixed)

Message Sent ID

Reported by: jpsb Owned by:
Milestone: 1.7 Priority: normal
Severity: normal Version: 1.6
Component: Messages Keywords: has-patch
Cc: jkudish

Description

Hi,

In function send of messaging component, in the file bp-messages-classes.php, the ID is placed like this: $ this-> id = $ wpdb-> insert_id on line 385;

But that’re getting the ID of the recipient, rather than the ID of the message sent.

To get the ID of the message correctly, the code $ this-> id = $ wpdb-> insert_id should appear on line 368 instead of line 385. Is there any reason for this?

Attachments (1)

4597.diff (1023 bytes) - added by jkudish 7 years ago.
Return the correct ID in the send() method of BP_Messages_Thread

Download all attachments as: .zip

Change History (5)

@jkudish
7 years ago

Return the correct ID in the send() method of BP_Messages_Thread

#1 @boonebgorges
7 years ago

  • Keywords has-patch added

#2 @jkudish
7 years ago

  • Cc jkudish added

#3 @boonebgorges
7 years ago

  • Milestone changed from Awaiting Review to 1.7

#4 @boonebgorges
7 years ago

  • Resolution set to fixed
  • Status changed from new to closed

(In [6413]) Return the correct item id from BP_Messages_Message::send()

Fixes #4597

Props jpsb, jkudish

Note: See TracTickets for help on using tickets.