[buddypress-trac] [BuddyPress Trac] #6161: Group Bulk Edit
buddypress-trac
noreply at wordpress.org
Sat Oct 3 17:13:20 UTC 2015
#6161: Group Bulk Edit
-------------------------------------------------+-------------------------
Reporter: sooskriszta | Owner:
Type: enhancement | Status: new
Priority: normal | Milestone: Future
Component: Component - Groups | Release
Severity: normal | Version:
Keywords: has-patch needs-testing needs- | Resolution:
refresh |
-------------------------------------------------+-------------------------
Changes (by imath):
* keywords: has-patch needs-testing => has-patch needs-testing needs-
refresh
* milestone: 2.4 => Future Release
Comment:
@web24coder thanks a lot for your patch.
I just gave a first look at it, and i have a few questions:
- why introduced: `BP_Groups_Member::get_all_members_groups()` ? Couldn't
`groups_get_group_members()` be used instead ?
- would it be possible for you to adapt the code to
[https://make.wordpress.org/core/handbook/best-practices/coding-
standards/php/ WordPress Coding Standards] ? Because this is something
we'll need to do, so it could save us some time :)
- If i understand well the code the form to bulk update the groups is
always shown, in your opinion would it be possible to use/extend WordPress
builtin WP_Posts_List_Table->inline_edit() method instead, so that the UI
is similar to WordPress one ?
- Finally (i know this is a detail but i'm trying :) ) Could you use
BuddyPress trunk to build your patch ? If you need help, i'll be happy
give you a hand.
And i'm sorry, I think we're too close to the end of 2.4.0 dev-cycle. So
let's take the time to work on this later. I agree it's an interesting
feature. It would be great if @dcavins could have a look at it.
--
Ticket URL: <https://buddypress.trac.wordpress.org/ticket/6161#comment:9>
BuddyPress Trac <http://buddypress.org/>
BuddyPress Trac
More information about the buddypress-trac
mailing list