[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                |
-------------------------------------+---------------------

Comment (by imath):

 In [changeset:"13269" 13269]:
 {{{
 #!CommitTicketReference repository="" revision="13269"
 Avoid 2 superfluous private message notification queries

 When viewing the single view of the thread, the potential notifications
 about it are marked as read into the `messages_action_conversation()`
 function. We don't need to do it again when looping into the thread
 messages.

 This was revealed by a database error which was appearing when a user who
 had received more than 1 message from two different threads was directly
 viewing the single view of one of the threads.

 Props mike80222, oztaser
 Closes https://github.com/buddypress/buddypress/pull/14
 See #8676 (trunk)
 }}}

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