[buddypress-trac] [BuddyPress Trac] #5336: Add "manage invitations" pane to group admin screen

buddypress-trac noreply at wordpress.org
Mon Jan 27 21:14:28 UTC 2014


#5336: Add "manage invitations" pane to group admin screen
-----------------------------------------+--------------------
 Reporter:  dcavins                      |       Owner:
     Type:  enhancement                  |      Status:  new
 Priority:  normal                       |   Milestone:  2.0
Component:  Administration               |     Version:  1.9.1
 Severity:  normal                       |  Resolution:
 Keywords:  has-patch reporter-feedback  |
-----------------------------------------+--------------------

Comment (by dcavins):

 I agree. I don't like adding new things that'll have to be maintained
 later. With the invites loop, I seem to remember that I couldn't use it
 because it is scoped by the group and by the user who is viewing the list
 (and this list needs to show all outstanding invitations for the group).
 I'll double-check that. Also, I'm not sure that `bp_group_has_invites()`
 handles pagination. I don't see any code for it in the class, but I could
 be missing it.

 I'm a little concerned about passing pagination to the
 `bp_group_has_membership_requests()` loop, too. It's looking for
 `$_REQUEST['num']` which seems a little too generic if there are multiple
 paginated loops on the same page. Maybe the better answer is to modify the
 class to use a more specifically named parameter so it can be used here as
 well as in the front-end templates.

 Thanks for thinking about this problem.

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


More information about the buddypress-trac mailing list