[buddypress-trac] [BuddyPress Trac] #8676: Database Errors for multiple notifications

[buddypress-trac]

#8676: Database Errors for multiple notifications
----------------------------------------+---------------------
 Reporter:  mike80222                   |       Owner:  (none)
     Type:  defect (bug)                |      Status:  new
 Priority:  normal                      |   Milestone:  10.3.0
Component:  Toolbar & Notifications     |     Version:  10.0.0
 Severity:  normal                      |  Resolution:
 Keywords:  has-patch needs-unit-tests  |
----------------------------------------+---------------------

Comment (by oztaser):

 Hi @imath,

 To see this PHP warning, send a some user private messages from two
 different user (that means two thread) and open the single view page one
 of the threads. `https://buddypress.test/members/test2/messages/view/5/`

 As I said my previous comment we are fetching other thread message ids in
 thread_loop_start action hook to mark message notifications as read but
 these message ids are not belongs to current thread so we're using empty
 array.

 I don't know how to write unit test for this situation I think we need to
 write some test for `BP_Notifications_Notification::update_id_list`
 function.

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