[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:  3.0
Component:  Toolbar & Notifications  |     Version:
 Severity:  normal                   |  Resolution:
 Keywords:  has-patch                |
-------------------------------------+------------------

Comment (by boonebgorges):

 In [changeset:"11851"]:
 {{{
 #!CommitTicketReference repository="" revision="11851"
 Notifications: User count query should be separately fetched and cached.

 Previously, notification counts were calculated by fetching the entire
 notification objects.

 Props m_uysl.
 See #7130.
 }}}

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