[buddypress-trac] [BuddyPress Trac] #7614: Group member count routine is bad

[buddypress-trac]

#7614: Group member count routine is bad
--------------------------+-----------------------
 Reporter:  boonebgorges  |       Owner:  imath
     Type:  enhancement   |      Status:  assigned
 Priority:  normal        |   Milestone:  11.0.0
Component:  Groups        |     Version:  1.2.3
 Severity:  normal        |  Resolution:
 Keywords:  needs-patch   |
--------------------------+-----------------------

Comment (by imath):

 Hi @dcavins & @sjregan

 Thanks a lot for your contributions. It's a complex issue. I need to point
 you to #8688 to add some context about why I've reopened this ticket. I've
 introduced the `count` parameter at this time to avoid populating group
 member extra data when it's not needed (like when counting members) as it
 caused performance issues on profile.wordpress.org.

 So I'm not very comfortable with your suggestion @dcavins ( using `'count'
 => false` ), because we'd probably be back to a poor performance situation
 when a group has many members like on profiles.wordpress.org.

 Although I understand @sjregan's need, I'd rather take the restrict Admin
 UI to active users only. By the way, reading your comment, I don't see why
 restricting the Admin UI has an impact about

 > The invitees receive the invitation email, complete their profile and
 immediately have the access to their required groups for their
 responsibilities within the business

 If the above process is automatic, why do you need the Admin UI for ?

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