[buddypress-trac] [BuddyPress Trac] #7105: Improve group management members screen

[buddypress-trac]

#7105: Improve group management members screen
--------------------------------+-----------------------------
 Reporter:  dcavins             |       Owner:
     Type:  enhancement         |      Status:  new
 Priority:  normal              |   Milestone:  Future Release
Component:  Component - Groups  |     Version:  1.1
 Severity:  normal              |  Resolution:
 Keywords:  has-patch           |
--------------------------------+-----------------------------
Changes (by r-a-y):

 * version:  2.5.0 => 1.1


Comment:

 Markup changes look fine to me.  Would it make sense to mirror the markup
 of the `/members/members-loop.php` file here?

 The switch to `bp_group_has_members()`, while it makes logical sense to do
 so, is actually more query-intensive as that utilizes
 `BP_Group_Members_Query`, which has to re-query for the group
 administrator / group moderator IDs and populate some group member data.
 Would like @boonebgorges to chime in here.

 Also, just to keep in mind that offereins has a patch to split up the
 admin.php template into separate template parts - #7079.  That patch would
 probably need to be refreshed as well.

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