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

buddypress-trac noreply at wordpress.org
Tue Sep 14 16:47:16 UTC 2021


#8426: Update queries when new message notifications mark as read
-------------------------------------+---------------------
 Reporter:  oztaser                  |       Owner:  (none)
     Type:  defect (bug)             |      Status:  new
 Priority:  normal                   |   Milestone:  10.0.0
Component:  Messages                 |     Version:
 Severity:  normal                   |  Resolution:
 Keywords:  has-patch needs-testing  |
-------------------------------------+---------------------

Comment (by oztaser):

 Hey, thanks for the update.


 > Shouldn't we also use bp_notifications_mark_notifications_by_item_ids()
 inside bp_messages_screen_conversation_mark_notifications()?

 Yes, we should. I've updated.

 > Maybe we should also improve bp_messages_message_delete_notifications()
 the same way?

 More performance improvements I liked that :) I've also updated the patch
 for this.

 I've added new functions called
 `bp_notifications_mark_notifications_by_ids` and
 `bp_notifications_delete_notifications_by_ids`. I thought these functions
 could be helpful.

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


More information about the buddypress-trac mailing list