[buddypress-trac] [BuddyPress Trac] #7130: bp_notifications_get_all_notifications_for_user gets all records and can cause memory leak

[buddypress-trac]

#7130: bp_notifications_get_all_notifications_for_user gets all records and can
cause memory leak
-------------------------------------+-----------------------------
 Reporter:  hberberoglu              |       Owner:
     Type:  defect (bug)             |      Status:  new
 Priority:  normal                   |   Milestone:  Future Release
Component:  Toolbar & Notifications  |     Version:
 Severity:  normal                   |  Resolution:
 Keywords:                           |
-------------------------------------+-----------------------------

Comment (by hberberoglu):

 I agree with @DJPaul. My thought was the same with him when I'm sending
 this ticket.

 On my side we've already solved this problem in some way but it required
 to write code in core BuddyPress which we never done before. We wrote code
 core of WordPress because of problems like this (big database, high
 traffic) before but this is the first time we writing to core BuddyPress.
 Actually we just wanted to get count of notifications of some user and
 show it on our mobile applications via rest api. We use default memory
 limit and didn't/don't want to increase it so it caused memory exhaustion.

 Btw maybe we can solve it by using bp_notifications_get_where_conditions
 but we didn't looked this filter yet. It is again not a good way I think.
 Yes it solves to writing code to core but it is not the best way.

 I hope it can solve in core.

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