[buddypress-trac] [BuddyPress Trac] #7175: Display and filter by group type on Dashboard > Groups

[buddypress-trac]

#7175: Display and filter by group type on Dashboard > Groups
-------------------------+------------------
 Reporter:  mercime      |       Owner:
     Type:  enhancement  |      Status:  new
 Priority:  normal       |   Milestone:  2.7
Component:  Groups       |     Version:
 Severity:  normal       |  Resolution:
 Keywords:               |
-------------------------+------------------

Comment (by dcavins):

 Replying to [comment:4 DJPaul]:
 > Since you asked:
 >
 > * bp_current_user_can vs current_user_can :)
 > * strict comparision operators please :)
 > * with in_array, please set the third parameter to true (good habit to
 get into) :)
 > * In column_content_group_type(), escape the `singular_name` label
 within that link. :)
 > * `$columns[ 'bp_group_type' ]` should be ` $columns['bp_group_type']`.
 The spacing is only needed if you're using an variable to access an array
 element, e.g. `$my_array[ $my_var ]`. :)
 > * In add_group_type_bulk_change_select(), please escape the all the
 things. :)
 >
 > :)

 Thanks for taking the time to read over the patch! That's very helpful.

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