[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