[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