[buddypress-trac] [BuddyPress Trac] #8426: Update queries when new message notifications mark as read

buddypress-trac noreply at wordpress.org
Sun Sep 19 15:58:15 UTC 2021


#8426: Update queries when new message notifications mark as read
-------------------------------------+---------------------
 Reporter:  oztaser                  |       Owner:  imath
     Type:  defect (bug)             |      Status:  closed
 Priority:  normal                   |   Milestone:  10.0.0
Component:  Toolbar & Notifications  |     Version:  1.9
 Severity:  normal                   |  Resolution:  fixed
 Keywords:  has-patch commit         |
-------------------------------------+---------------------
Changes (by imath):

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


Comment:

 In [changeset:"13112" 13112]:
 {{{
 #!CommitTicketReference repository="" revision="13112"
 Improve SQL performance when updating/deleting a list of notifications

 Instead of looping into a list of notifications to run an update/delete
 query for each notification, update/delete a list of notifications using a
 single query. Use primarly this change for message thread notifications
 and bulk notification actions.

 Mainly introduces 2 new static methods to `BP_Notifications_Notification`:
 - `BP_Notifications_Notification::update_id_list()` let you update a list
 of notifications using their notification IDs or component item IDs.
 - `BP_Notifications_Notification::delete_id_list()` let you delete a list
 of notifications using their notification IDs or component item IDs.

 Props oztaser

 Fixes #8426
 }}}

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


More information about the buddypress-trac mailing list