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

[buddypress-trac]

#8676: Database Errors for multiple notifications
-------------------------------------+---------------------
 Reporter:  mike80222                |       Owner:  imath
     Type:  defect (bug)             |      Status:  closed
 Priority:  normal                   |   Milestone:  10.3.0
Component:  Toolbar & Notifications  |     Version:  10.0.0
 Severity:  normal                   |  Resolution:  fixed
 Keywords:  has-patch                |
-------------------------------------+---------------------
Changes (by imath):

 * owner:  (none) => imath
 * status:  new => closed
 * resolution:   => fixed


Comment:

 In [changeset:"13270" 13270]:
 {{{
 #!CommitTicketReference repository="" revision="13270"
 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

 Fixes #8676 (10.0 branch)
 }}}

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